jauco / jester

Get your project tested and out there with minimal fuss.
https://trello.com/b/GLFKT2X4/jester
BSD 3-Clause "New" or "Revised" License
3 stars 3 forks source link

Add jsx support #13

Closed jauco closed 9 years ago

barbarah commented 9 years ago

Om dit werkend te krijgen moest ik in jester.json het volgende aanpassen:

 "entryGlob": "features/*/feature.@(js|jsx)",

Kan dit ergens in een readme?

barbarah commented 9 years ago

Er wordt ook een (featureName).chunk.js gegenereerd bij jsx bestanden, waar is dit voor?

barbarah commented 9 years ago

code ziet er verder wel ok uit!

barbarah commented 9 years ago

*.test.jsx werkt niet

jauco commented 9 years ago

re: chunk. Ik denk dat jsx ergens require.ensure gebruikt.

odegroot commented 9 years ago

[Done] npm-shrinkwrap.json is nu corrupt: ontbrekende komma's. Voorbeeld:

  "from": "https://registry.npmjs.org/karma-phantomjs-launcher/-/karma-phantomjs-launcher-0.1.2.tgz"
  "dependencies": {

npm install faalt daardoor nu.

$ npm install
npm ERR! install Couldn't read dependencies
npm ERR! SyntaxError: Unexpected string
...
odegroot commented 9 years ago

Testinstructies: runnen en checken of ik een werkende source map voor een jsx-file kan zien.

odegroot commented 9 years ago

Ik zie feature..js (dubbele punt) langskomen in de output. Moeten we daar iets mee?

PhantomJS 1.9.7 (Linux) feature testsuite 1 adds the properties of the base object to the destination object FAILED
    ReferenceError: Can't find variable: React
        at translate (/home/oscar.degroot/Workspace/jsxtest/build/karma/feature..js:65)
odegroot commented 9 years ago

Bij jester-watch wordt ESLint losgelaten op het jsx-bestand, wat uiteraard faalt.