Skip to content

Commit aa6e8a5

Browse files
committed
docs: update testing guide for copy command with new test cases
1 parent 0c2eeb2 commit aa6e8a5

2 files changed

Lines changed: 30 additions & 18 deletions

File tree

.github/workflows/magento-compatibility.yml

Lines changed: 28 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -150,19 +150,23 @@ jobs:
150150
- name: Test Copy Command Functionality
151151
working-directory: magento2
152152
run: |
153-
echo "Test: Copy frontend file to frontend theme (dry-run):"
153+
echo "Test 1: Copy frontend template to frontend theme (dry-run):"
154154
bin/magento mageforge:theme:copy-from-vendor \
155-
vendor/magento/module-catalog/view/frontend/templates/product/list.phtml \
155+
vendor/magento/module-theme/view/frontend/templates/page/copyright.phtml \
156156
Magento/luma \
157157
--dry-run
158+
echo "✓ Frontend to frontend mapping works"
159+
echo ""
158160
159-
echo "Test: Copy base file to frontend theme (dry-run):"
161+
echo "Test 2: Copy base template to frontend theme (dry-run):"
160162
bin/magento mageforge:theme:copy-from-vendor \
161-
vendor/magento/module-theme/view/base/web/css/print.css \
163+
vendor/magento/module-theme/view/base/templates/root.phtml \
162164
Magento/blank \
163165
--dry-run
166+
echo "✓ Base to frontend mapping works"
167+
echo ""
164168
165-
echo "Test: Verify non-view file is rejected:"
169+
echo "Test 3: Verify non-view file is rejected:"
166170
if bin/magento mageforge:theme:copy-from-vendor \
167171
vendor/magento/module-catalog/etc/di.xml \
168172
Magento/luma \
@@ -172,19 +176,21 @@ jobs:
172176
echo "✗ Non-view file validation failed"
173177
exit 1
174178
fi
179+
echo ""
175180
176-
echo "Test: Verify cross-area mapping is rejected (frontend -> adminhtml):"
181+
echo "Test 4: Verify cross-area mapping is rejected (frontend -> adminhtml):"
177182
if bin/magento mageforge:theme:copy-from-vendor \
178-
vendor/magento/module-catalog/view/frontend/templates/product/list.phtml \
183+
vendor/magento/module-theme/view/frontend/templates/page/copyright.phtml \
179184
Magento/backend \
180185
--dry-run 2>&1 | grep -q "Cannot map file from area"; then
181186
echo "✓ Cross-area mapping correctly rejected"
182187
else
183188
echo "✗ Cross-area validation failed"
184189
exit 1
185190
fi
191+
echo ""
186192
187-
echo "All copy command tests passed!"
193+
echo "All copy command tests passed!"
188194
189195
- name: Test Summary
190196
run: |
@@ -322,19 +328,23 @@ jobs:
322328
- name: Test Copy Command Functionality
323329
working-directory: magento2
324330
run: |
325-
echo "Test: Copy frontend file to frontend theme (dry-run):"
331+
echo "Test 1: Copy frontend template to frontend theme (dry-run):"
326332
bin/magento mageforge:theme:copy-from-vendor \
327-
vendor/magento/module-catalog/view/frontend/templates/product/list.phtml \
333+
vendor/magento/module-theme/view/frontend/templates/page/copyright.phtml \
328334
Magento/luma \
329335
--dry-run
336+
echo "✓ Frontend to frontend mapping works"
337+
echo ""
330338
331-
echo "Test: Copy base file to frontend theme (dry-run):"
339+
echo "Test 2: Copy base template to frontend theme (dry-run):"
332340
bin/magento mageforge:theme:copy-from-vendor \
333-
vendor/magento/module-theme/view/base/web/css/print.css \
341+
vendor/magento/module-theme/view/base/templates/root.phtml \
334342
Magento/blank \
335343
--dry-run
344+
echo "✓ Base to frontend mapping works"
345+
echo ""
336346
337-
echo "Test: Verify non-view file is rejected:"
347+
echo "Test 3: Verify non-view file is rejected:"
338348
if bin/magento mageforge:theme:copy-from-vendor \
339349
vendor/magento/module-catalog/etc/di.xml \
340350
Magento/luma \
@@ -344,19 +354,21 @@ jobs:
344354
echo "✗ Non-view file validation failed"
345355
exit 1
346356
fi
357+
echo ""
347358
348-
echo "Test: Verify cross-area mapping is rejected (frontend -> adminhtml):"
359+
echo "Test 4: Verify cross-area mapping is rejected (frontend -> adminhtml):"
349360
if bin/magento mageforge:theme:copy-from-vendor \
350-
vendor/magento/module-catalog/view/frontend/templates/product/list.phtml \
361+
vendor/magento/module-theme/view/frontend/templates/page/copyright.phtml \
351362
Magento/backend \
352363
--dry-run 2>&1 | grep -q "Cannot map file from area"; then
353364
echo "✓ Cross-area mapping correctly rejected"
354365
else
355366
echo "✗ Cross-area validation failed"
356367
exit 1
357368
fi
369+
echo ""
358370
359-
echo "All copy command tests passed!"
371+
echo "All copy command tests passed!"
360372
361373
- name: Test Summary
362374
run: |

docs/testing_copy_command.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -239,13 +239,13 @@ The command should be tested in CI/CD pipeline:
239239
run: |
240240
# Test basic functionality
241241
bin/magento mageforge:theme:copy-from-vendor --help
242-
242+
243243
# Test dry-run mode
244244
bin/magento mageforge:theme:copy-from-vendor \
245245
vendor/magento/module-catalog/view/frontend/templates/product/list.phtml \
246246
Magento/luma \
247247
--dry-run
248-
248+
249249
# Test error handling
250250
if bin/magento mageforge:theme:copy-from-vendor \
251251
vendor/magento/module-catalog/etc/di.xml \

0 commit comments

Comments
 (0)