Skip to content

Commit

Permalink
correct hash names
Browse files Browse the repository at this point in the history
  • Loading branch information
aarbi committed Nov 10, 2021
1 parent c802616 commit 31bd616
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 11 deletions.
11 changes: 6 additions & 5 deletions test/features/mix.js
Original file line number Diff line number Diff line change
Expand Up @@ -55,20 +55,21 @@ test('it resolves image- and font-urls and distinguishes between them even if we
// Then we expect the css to be built
t.true(File.exists(`test/fixtures/app/dist/css/font-and-image.css`));
// Along with the referred image in the images folder
t.true(File.exists(`test/fixtures/app/dist/images/img.66162.svg`));
t.true(File.exists(`test/fixtures/app/dist/images/img.66162863.svg`));
// And the referred fonts in the fonts folder
t.true(File.exists(`test/fixtures/app/dist/fonts/font.svg`));
t.true(File.exists(`test/fixtures/app/dist/fonts/awesome.svg`));
// And we expect the image NOT to be in the fonts folder:
t.false(File.exists(`test/fixtures/app/dist/fonts/img.svg`));
// And the fonts NOT to be in the image folder
t.false(File.exists(`test/fixtures/app/dist/images/font.66162.svg`));
t.false(File.exists(`test/fixtures/app/dist/images/awesome.66162.svg`));
t.false(File.exists(`test/fixtures/app/dist/images/font.66162863.svg`));
t.false(File.exists(`test/fixtures/app/dist/images/awesome.66162863.svg`));
});

test('it accepts images with the same file name but with different paths', async t => {
mix.js(`test/fixtures/app/src/js/app-import-images-same-file-name.js`, 'js');
await webpack.compile();
t.true(File.exists(`test/fixtures/app/dist/images/img.66162.svg`));
t.true(File.exists(`test/fixtures/app/dist/images/img.9e0d3.svg`));

t.true(File.exists(`test/fixtures/app/dist/images/img.66162863.svg`));
t.true(File.exists(`test/fixtures/app/dist/images/img.9e0d380b.svg`));
});
8 changes: 4 additions & 4 deletions test/features/preprocessors.js
Original file line number Diff line number Diff line change
Expand Up @@ -251,11 +251,11 @@ test('Sass url resolution can be configured per-file', async t => {
t.true(File.exists(`test/fixtures/app/dist/css/font-and-image.css`));
t.true(File.exists(`test/fixtures/app/dist/css/image.css`));

t.false(File.exists(`test/fixtures/app/dist/images/img.66162.svg`));
t.true(File.exists(`test/fixtures/app/dist/images/img2.66162.svg`));
t.false(File.exists(`test/fixtures/app/dist/images/img.66162863.svg`));
t.true(File.exists(`test/fixtures/app/dist/images/img2.66162863.svg`));

t.false(File.exists(`test/fixtures/app/dist/fonts/font.66162.svg`));
t.false(File.exists(`test/fixtures/app/dist/fonts/awesome.66162.svg`));
t.false(File.exists(`test/fixtures/app/dist/fonts/font.66162863.svg`));
t.false(File.exists(`test/fixtures/app/dist/fonts/awesome.66162863.svg`));
});

test('Sass url resolution can be disabled: globally (before)', async t => {
Expand Down
4 changes: 2 additions & 2 deletions test/features/setResourceRoot.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,13 @@ test('mix.setResourceRoot() rewrites processed asset urls', async t => {
await webpack.compile();

t.true(File.exists(`test/fixtures/app/dist/css/app-and-image.css`));
t.true(File.exists(`test/fixtures/app/dist/images/img.66162.svg`));
t.true(File.exists(`test/fixtures/app/dist/images/img.66162863.svg`));

assert.fileMatchesCss(
`test/fixtures/app/dist/css/app-and-image.css`,
`.app {
color: red;
background-image: url(https://www.example.com/images/img.svg?66162863e7583212a5d4fd6cdc2426ed);
background-image: url(https://www.example.com/images/img.66162863.svg?66162863);
}`,
t
);
Expand Down

0 comments on commit 31bd616

Please sign in to comment.