frenic / csstype

Strict TypeScript and Flow types for style based on MDN data
MIT License
1.73k stars 72 forks source link

Test failing on ppc64le #136

Open gitssinghal opened 3 years ago

gitssinghal commented 3 years ago

Hi Team,

I am trying to build csstype 3.0.8 (with npm 6.14.13 and node v12.22.3) package on ppc64le and the build gets successful. However one of the test case is failing as below

FAIL tests/dist.flow.ts ● it detects errors

TypeError: The "file" argument must be of type string. Received nullTypeError [ERR_INVALID_ARG_TYPE]: The "file" argument must be of type string. Received null

   6 |
   7 | test('it detects errors', () => {
>  8 |   const { stdout: output } = spawnSync(
     |                              ^
   9 |     flow,
  10 |     ['check', path.resolve(__dirname, '__fixtures__/typecheck.js'), '--json'],
  11 |     {

  at Object.<anonymous> (__tests__/dist.flow.ts:8:30)

I tried to include json for string match in jest.config.js and have checked for the presence of typecheck.js in the said folder.

But probably I am missing something else. Please help here. Please note that the said test case passes on intel machine.

Regards, Swati Singhal

frenic commented 3 years ago

Does this command $ npx flow check __tests__/__fixtures__/typecheck.js work?

gitssinghal commented 3 years ago

thank you for the response. The command did not succeed. flow was missing. I added flow using "yarn" post that it gave me below error

"Platform not supported."

Hence trying to build flow from source code, using below

https://github.com/facebook/flow

it requires me to have opam which is again not platform supported. hence building that also. Let me if that's the right approach.

frenic commented 3 years ago

If Flow isn't supported on ppc64le then I can't help you, sorry. You should reach out to https://github.com/facebook/flow for guidance.