Skip to content
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

don't output resource error, Fix spm#1004 #6

Merged
merged 2 commits into from
Sep 11, 2014
Merged

Conversation

sorrycc
Copy link

@sorrycc sorrycc commented Sep 11, 2014

@sorrycc
Copy link
Author

sorrycc commented Sep 11, 2014

当发起请求 about:blank 的时候,phantomjs 认为无法加载,输出 resoruce error 信息,然后会导致 coverage 信息解析出错。

@afc163
Copy link
Owner

afc163 commented Sep 11, 2014

这样删掉有点粗暴啊

@sorrycc
Copy link
Author

sorrycc commented Sep 11, 2014

改成解析的时候删除 Error loading resource 的信息了。

afc163 added a commit that referenced this pull request Sep 11, 2014
don't output resource error, Fix spm#1004
@afc163 afc163 merged commit 598a7ff into afc163:master Sep 11, 2014
@sorrycc
Copy link
Author

sorrycc commented Sep 11, 2014

更新个版本吧。

@afc163
Copy link
Owner

afc163 commented Sep 11, 2014

0.3.1 published

@afc163
Copy link
Owner

afc163 commented Jan 20, 2015

https://travis-ci.org/aralejs/dialog#L340 这里出现了类似的问题:

undefined:2968

}Unsafe JavaScript attempt to access frame with URL about:blank from frame wit

^

SyntaxError: Unexpected token U

at Object.parse (native)

at cleanJSON (/home/travis/build/aralejs/dialog/node_modules/spm/node_modules/mocha-browser2/index.js:183:15)

at EventEmitter.<anonymous> (/home/travis/build/aralejs/dialog/node_modules/spm/node_modules/mocha-browser2/index.js:164:12)

at EventEmitter.emit (events.js:92:17)

at Socket.<anonymous> (/home/travis/build/aralejs/dialog/node_modules/spm/node_modules/mocha-browser2/index.js:76:15)

at Socket.emit (events.js:117:20)

at _stream_readable.js:944:16

at process._tickCallback (node.js:442:13)

PS:重构过的并非解析 json ,应该不会有问题。

现在是这种 log 不知道有什么办法可以干掉。

afc163 added a commit that referenced this pull request Jan 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants