Skip to content

Commit

Permalink
Remove indentedSyntax option from tests
Browse files Browse the repository at this point in the history
Thus we are testing #196 implicitly
  • Loading branch information
jhnns committed Mar 9, 2016
1 parent 97dfe11 commit a114cca
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions test/index.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,10 @@ describe('sass-loader', function () {

it('should override sassLoader config with loader query', function () {
var expectedCss = readCss('sass', 'language');
var sassFile = pathToSassFile('sass', 'language');
var webpackConfig = Object.assign({}, {
entry: sassFile,
entry: 'raw!' + pathToSassLoader + '?indentedSyntax!' + path.join(__dirname, 'sass', 'language.sass'),
sassLoader: {
// Incorrect setting here should be overridden by loader query string given by
// pathToSassFile()
// Incorrect setting here should be overridden by loader query
indentedSyntax: false
}
});
Expand Down Expand Up @@ -262,5 +260,5 @@ function testSync(name, id, config) {
}

function pathToSassFile(ext, id) {
return 'raw!' + pathToSassLoader + '?' + (ext === 'sass'? 'indentedSyntax' : '') + '!' + path.join(__dirname, ext, id + '.' + ext);
return 'raw!' + pathToSassLoader + '!' + path.join(__dirname, ext, id + '.' + ext);
}

0 comments on commit a114cca

Please # to comment.