mikemaccana / agave

Cleaner, simpler JavaScript for ES6
MIT License
114 stars 10 forks source link

npm test doesn't work #12

Open azachar opened 7 years ago

azachar commented 7 years ago

Hello, I like your idea and want to fork it in order to provide an api that allows to add custom policies objects instead of just names, but running npm test is on your current code base is not working..

I am getting this results: (Also, I neeedd to use this form var agave = require('agave')('av'); in your test)


"fontSrc": [
           "data:"
           "fonts.googleapis.com"
           "fonts.gstatic.com"
      -    "themes.googleusercontent.com"
           "use.typekit.net"
         ]

         "frameSrc": [
           "js.stripe.com"
         "reportUri": "/csp-violation"
         "scriptSrc": [
           "'self'"
           "'unsafe-eval'"
      -    "analytics.twitter.com"
      -    "api.stripe.com"
           "cdn.mxpnl.com"
      -    "cdn.syndication.twimg.com"
           "js.stripe.com"
           "platform.twitter.com"
      -    "static.ads-twitter.com"
           "syndication.twitter.com"
           "use.typekit.net"
         ]
         "styleSrc": [

Please, could you please fix your code so I can provide my patches?

Thanks, Andrej

rousan commented 6 years ago

I sent a PR against this issue.

mikemaccana commented 6 years ago

The test output you've provided doesn't look like anything to with agave: image

Did you mean to file a bug with https://github.com/certsimple/csp-by-api instead?

Also all the tests pass there too (see the CI output in the README)