-
Notifications
You must be signed in to change notification settings - Fork 106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ISSUE-79: Introduce smoke tests #81
Conversation
e7e658e
to
63b5a30
Compare
eb56133
to
1335fb9
Compare
src/providers/reader.spec.ts
Outdated
@@ -153,7 +154,7 @@ describe('Providers → Reader', () => { | |||
|
|||
const entry = tests.getFileEntry(false /** dot */); | |||
|
|||
const expected: string = path.join(process.cwd(), 'fixtures/file.txt'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are results always converted to forward slashes now?
This makes matching slashes paths that are returned harder since Node will normalize slashes in Windows as backslashes. My tests have been failing for a while and I finally found this as the root cause.
As a workaround I've results.map(path.normalize)
.
What is the purpose of this pull request?
This is an introduction to smoke tests (#79) for the
fast-glob
package.What changes did you make? (Give an overview)
transform
action