Skip to content

Update browser_tab.py#426

Open
Japan10 wants to merge 16 commits intoscrapinghub:masterfrom
Japan10:master
Open

Update browser_tab.py#426
Japan10 wants to merge 16 commits intoscrapinghub:masterfrom
Japan10:master

Conversation

@Japan10
Copy link
Copy Markdown

@Japan10 Japan10 commented Mar 24, 2016

Line: 915
Change "user-agent" to "User-Agent"

Line:915
Change "user-agent" to "User-Agent"
@codecov-io
Copy link
Copy Markdown

Current coverage is 85.62%

Merging #426 into master will decrease coverage by -0.09% as of 7355a3b

@@            master    #426   diff @@
======================================
  Files           39      39       
  Stmts         4627    4627       
  Branches       633     633       
  Methods          0       0       
======================================
- Hit           3966    3962     -4
- Partial        156     159     +3
- Missed         505     506     +1

Review entire Coverage Diff as of 7355a3b

Powered by Codecov. Updated on successful CI builds.

@dvdbng
Copy link
Copy Markdown
Contributor

dvdbng commented Mar 24, 2016

Hi @Genius001 thanks for the pull request! There are some test failing because they have "user-agent" instead of "User-Agent" hardcoded in them, would you mind fixing that as well?

@Japan10
Copy link
Copy Markdown
Author

Japan10 commented Mar 24, 2016

Hi, @Youwotma

I noticed if change all 'user-agent' to 'User-Agent' in tests/test_execute.py that will cause another problem, And test will failing.

The error log https://travis-ci.org/scrapinghub/splash/jobs/118187194#L2146

I could not figure out how to resolve it.
If I fix all these headers key(cache-control, accept-encoding,accept-language) to case sensitive.
That could be affect other test file?

But this time I thought whether is it necessary to do this or keep original?

If it is necessary, workaround?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants