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.

Sign up for GitHub

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

Jump to bottom

Initial set of tests for Content Security Policy #1120

Merged
merged 39 commits into from Jul 31, 2014
Merged

Initial set of tests for Content Security Policy #1120

merged 39 commits into from Jul 31, 2014

Conversation

hillbrad
Copy link
Contributor

This checkin contains:

  1. An initial set of tests and supporting files for Content Security Policy
  2. A README (html+css) on how to write tests
  3. A python handler support script for stashing and retrieving CSP violation reports

@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/2137

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@hillbrad
better error handling for empty stash
43ca658
@hillbrad
fixed typo wihtout->without
10cad80
@hillbrad
add distinct tests for eval blocking and reporting
a36ee84
@hillbrad
clarify reporting test description
808d56b
@hillbrad
use assert_unreached for tests which should not run according to policy
82abb16
@hillbrad
start async tests and use step(assert_unreachable) so test count is s…
f269d3c 
…ame whether browser passes or fails
@hillbrad
cleanup handling of asserts in checking for report existence and cont…
2417fc0 
…ents
@hillbrad
remove dead, unreachable code not relevant to test case
4a415ea
@hillbrad
use assert_unreached
db0caba
@hillbrad
removed trailing whitespace
2e8805f
@hillbrad
removed console.log statements
49d0bb6
@hillbrad
whitespace
0f148c6
@hillbrad
insignificant code style issue repeating an built-in identifier 2x in…
50bf0f1 
… one-liner
@hillbrad
whitespace
ade65f4
@hillbrad
whitespace
4c95a38
@hillbrad
restructured xhr event handler to user step_func
2ea610b
@hillbrad
whitespace
16b96fc
@hillbrad
get timeout values for report checking from caller
bdc98c7
@hillbrad
parse timeout as float, use json.dumps instead of string concat
e1d1124
@hillbrad
use request.body instead of raw_input
c97c483
@hillbrad
whitespace
b1e5064
@hillbrad
whitespace
14a43f9
@hillbrad
whitespace
606bc30
@hillbrad
whitespace
20a8d31
@hillbrad
re-use of location variable
a218def
jgraham added a commit that referenced this pull request Jul 31, 2014
@jgraham
Merge pull request #1120 from hillbrad/hillbrad/content-security-poli…
1dab557 
…cy_001

Initial set of tests for Content Security Policy
@jgraham jgraham merged commit 1dab557 into web-platform-tests:master Jul 31, 2014
gsnedders added a commit that referenced this pull request Mar 28, 2017
@gsnedders
Merge pull request #1120 from Ms2ger/lint
382148f 
Lint on Travis.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
content-security-policy wg-webappsec
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@hillbrad @hoppipolla-critic-bot @jgraham