@@ -976,7 +976,7 @@ public static function mediaImportImageFixtureRollback()
976
976
977
977
/** @var \Magento\Framework\Filesystem\Directory\Write $varDirectory */
978
978
$ varDirectory = $ fileSystem ->getDirectoryWrite (DirectoryList::VAR_DIR );
979
- $ varDirectory ->delete ('import ' . DIRECTORY_SEPARATOR . ' images ' );
979
+ $ varDirectory ->delete ('import ' );
980
980
$ mediaDirectory ->delete ('catalog ' );
981
981
}
982
982
@@ -2626,43 +2626,33 @@ public function testImagesAreHiddenAfterImport(): void
2626
2626
$ actualAllProductImages = [];
2627
2627
$ product = $ this ->getProductBySku ('simple ' );
2628
2628
2629
- // Check that new images are imported and existing image is disabled after import
2629
+ // Check that new images are imported and existing image is disabled after import
2630
2630
$ productMediaData = $ product ->getData ('media_gallery ' );
2631
2631
2632
- if ( is_array ( $ productMediaData ['images ' ])) {
2633
- $ allProductImages = $ productMediaData ['images ' ];
2634
- $ this ->assertCount (3 , $ allProductImages , 'Images are imported incorrect ' );
2632
+ $ this -> assertNotEmpty ( $ productMediaData ['images ' ]);
2633
+ $ allProductImages = $ productMediaData ['images ' ];
2634
+ $ this ->assertCount (3 , $ allProductImages , 'Images are imported incorrect ' );
2635
2635
2636
- foreach ($ allProductImages as $ image ) {
2637
- $ actualAllProductImages [] = [
2638
- 'file ' => $ image ['file ' ],
2639
- 'label ' => $ image ['label ' ],
2640
- 'disabled ' => $ image ['disabled ' ],
2641
- ];
2642
- }
2636
+ foreach ($ allProductImages as $ image ) {
2637
+ $ actualAllProductImages [] = [
2638
+ 'file ' => $ image ['file ' ],
2639
+ 'label ' => $ image ['label ' ],
2640
+ 'disabled ' => $ image ['disabled ' ],
2641
+ ];
2643
2642
}
2644
2643
2645
2644
$ this ->assertEquals (
2646
2645
$ expectedAllProductImages ,
2647
2646
$ actualAllProductImages ,
2648
- 'Images statuses are incorrect after import '
2647
+ 'Images are incorrect after import '
2649
2648
);
2650
2649
2651
- // Check that on storefront only enabled images are shown
2652
- $ actualActiveImages = array_values ($ product ->getMediaGalleryImages ()->getItems ());
2653
- $ this ->assertCount (2 , $ actualActiveImages );
2654
-
2655
- foreach ($ actualActiveImages as $ actualActiveImage ) {
2656
- $ this ->assertNotEquals (
2657
- $ expectedHiddenImage ['file ' ],
2658
- $ actualActiveImage ->getFile (),
2659
- 'Image should be hidden after import '
2660
- );
2661
- $ this ->assertNotEquals (
2662
- $ expectedHiddenImage ['label ' ],
2663
- $ actualActiveImage ->getLabel (),
2664
- 'Image should be hidden after import '
2665
- );
2666
- }
2650
+ // Check that on storefront only enabled images are shown
2651
+ $ actualActiveImages = $ product ->getMediaGalleryImages ();
2652
+ $ this ->assertSame (
2653
+ $ expectedActiveImages ,
2654
+ $ actualActiveImages ->toArray (['file ' , 'label ' , 'disabled ' ])['items ' ],
2655
+ 'Hidden image is present on frontend after import '
2656
+ );
2667
2657
}
2668
2658
}
0 commit comments