Closed 0xorial closed 7 months ago
ok, so there's a few issues here.
the crlf thing is because on windows, you need to (before cloning) configure git with git config --global core.autocrlf true
.
For the node
thing, what happens if you run npx tap test/*.js
?
For the node thing, what happens if you run npx tap test/*.js?
Then the tests seem to run: 294 passing, 9 pending.
the crlf thing is because on windows, you need to (before cloning) configure git with git config --global core.autocrlf true.
Why don't you put .gitattributes in the repo?
Anyhow, after that I get the following:
PS C:\Users\user\repos\LIS\tape> npm run test
npm WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.
> tape@5.6.1 pretest C:\Users\user\repos\LIS\tape
> npm run lint
npm WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.
npm WARN config auth-type This method of SSO/SAML/OAuth is deprecated and will be removed in
npm WARN config a future version of npm in favor of web-based login.
npm WARN config cache-max This option has been deprecated in favor of `--prefer-online`
npm WARN config cache-min This option has been deprecated in favor of `--prefer-offline`.
npm WARN config optional Use `--omit=optional` to exclude optional dependencies, or
npm WARN config `--include=optional` to include them.
npm WARN config
npm WARN config Default value does install optional deps unless otherwise omitted.
npm WARN config shrinkwrap Use the --package-lock setting instead.
npm WARN config sso-poll-frequency The --auth-type method of SSO/SAML/OAuth will be removed in a future
npm WARN config version of npm in favor of web-based login.
npm WARN config sso-type The --auth-type method of SSO/SAML/OAuth will be removed in a future
npm WARN config version of npm in favor of web-based login.
npm WARN config tmp This setting is no longer used. npm stores temporary files in a special
npm WARN config location in the cache, and they are managed by
npm WARN config [`cacache`](http://npm.im/cacache).
> tape@5.6.1 prelint C:\Users\user\repos\LIS\tape
> npm-run-posix-or-windows eclint
npm WARN config global `--global`, `--local` are deprecated. Use `--location=global` instead.
npm WARN config auth-type This method of SSO/SAML/OAuth is deprecated and will be removed in
npm WARN config a future version of npm in favor of web-based login.
npm WARN config cache-max This option has been deprecated in favor of `--prefer-online`
npm WARN config cache-min This option has been deprecated in favor of `--prefer-offline`.
npm WARN config optional Use `--omit=optional` to exclude optional dependencies, or
npm WARN config `--include=optional` to include them.
npm WARN config
npm WARN config Default value does install optional deps unless otherwise omitted.
npm WARN config shrinkwrap Use the --package-lock setting instead.
npm WARN config sso-poll-frequency The --auth-type method of SSO/SAML/OAuth will be removed in a future
npm WARN config version of npm in favor of web-based login.
npm WARN config sso-type The --auth-type method of SSO/SAML/OAuth will be removed in a future
npm WARN config version of npm in favor of web-based login.
npm WARN config tmp This setting is no longer used. npm stores temporary files in a special
npm WARN config location in the cache, and they are managed by
npm WARN config [`cacache`](http://npm.im/cacache).
> tape@5.6.1 eclint:windows C:\Users\user\repos\LIS\tape
> eclint check *.js
index.js
015:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
016:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
018:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
019:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
020:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
021:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
022:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
023:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
024:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
025:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
027:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
028:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
029:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
030:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
032:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
033:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
034:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
036:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
037:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
039:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
040:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
042:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
043:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
044:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
046:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
047:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
048:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
049:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
050:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
051:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
052:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
053:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
054:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
056:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
057:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
058:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
060:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
061:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
062:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
064:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
066:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
070:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
071:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
072:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
073:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
075:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
076:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
077:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
079:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
080:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
081:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
082:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
083:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
084:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
085:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
086:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
088:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
089:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
090:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
091:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
093:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
095:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
096:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
097:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
099:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
100:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
101:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
103:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
104:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
105:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
107:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
108:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
109:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
110:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
111:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
112:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
113:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
114:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
115:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
116:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
118:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
120:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
124:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
125:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
126:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
127:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
128:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
129:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
130:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
132:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
133:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
134:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
135:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
136:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
137:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
138:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
139:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
140:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
141:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
142:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
143:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
145:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
146:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
147:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
148:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
149:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
151:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
152:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
154:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
155:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
156:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
157:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
158:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
160:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
161:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
162:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
163:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
164:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
165:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
166:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
167:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
168:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
169:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
171:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
172:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
173:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
175:01 ❌ invalid indent style: tab, expected: space
(EditorConfig indent_style https://goo.gl/8Qkrbr)
npm ERR! code ELIFECYCLE
npm ERR! errno 4294967295
npm ERR! tape@5.6.1 eclint:windows: `eclint check *.js`
npm ERR! Exit status 4294967295
npm ERR!
npm ERR! Failed at the tape@5.6.1 eclint:windows script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR! C:\Users\user\AppData\Roaming\npm-cache\_logs\2022-10-17T21_28_49_695Z-debug.log
Error: Command failed: npm run eclint:windows
at checkExecSyncError (node:child_process:828:11)
at Object.execSync (node:child_process:899:15)
at Object.<anonymous> (C:\Users\user\repos\LIS\tape\node_modules\npm-run-posix-or-windows\bin\cli.js:18:19)
at Module._compile (node:internal/modules/cjs/loader:1105:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
at Module.load (node:internal/modules/cjs/loader:981:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:77:12)
at node:internal/main/run_main_module:17:47 {
status: 4294967295,
signal: null,
output: [ null, null, null ],
pid: 21228,
stdout: null,
stderr: null
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! tape@5.6.1 prelint: `npm-run-posix-or-windows eclint`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the tape@5.6.1 prelint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR! C:\Users\user\AppData\Roaming\npm-cache\_logs\2022-10-17T21_28_49_727Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! tape@5.6.1 pretest: `npm run lint`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the tape@5.6.1 pretest script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR! C:\Users\user\AppData\Roaming\npm-cache\_logs\2022-10-17T21_28_49_770Z-debug.log
I could put gitattributes in the repo, but then i'd have to manually add a file to all of my repos, and that still wouldn't fix the underlying problem of Windows line endings for devs - until they learn to do this globally, at which point the per-repo file isn't needed anymore.
The --global
warning means you have a broken version of npm 8; you may want to upgrade node/npm.
Good catch on the eclint
checks; npm run eclint:windows
fails although npm run eclint
does not; I'll fix that momentarily.
If npm run tests-only
also fails, then that suggests that nyc
is unable to find node on windows. I'd love to figure out how to solve that; that would probably unblock adding windows tests to CI.
I managed to get npm run tests-only
running by doing npm i nyc
. Seemingly otherwise it uses nyc@8, which doesn't work whereas nyc@15 works. (with a minor tweak of quotes : "tests-only": "nyc tap \"test/*.js\"",
Any comments on that?
And out of curiosity - why doesn't tape use itself to run tests?
nyc 15 doesn't work on the older browser engines we support.
Tape using itself to run tests is a bit strange imo; I think it would increase the chances in a bug being obscured.
I believe this was fixed by #602
Tried running tape tests on windows. Got errors. Tried a bunch of stuff so just putting console output here: