Thanks to visit codestin.com
Credit goes to github.com

Skip to content

improve BrowserKit test coverage p1 #16549

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

Closed
wants to merge 1 commit into from
Closed

improve BrowserKit test coverage p1 #16549

wants to merge 1 commit into from

Conversation

eventhorizonpl
Copy link

Hi,

This PR improves BrowserKit test coverage.

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
License MIT

improve BrowserKit test coverage p2

improve BrowserKit test coverage p3

improve BrowserKit test coverage p4

improve BrowserKit test coverage p5
@stof
Copy link
Member

stof commented Nov 16, 2015

shouldn't this be merged into older branches to cover it in all relevant versions ?

@eventhorizonpl
Copy link
Author

@stof

You mean all supported version? I can base my next PR's on 2.7 if you want.

@fabpot
Copy link
Member

fabpot commented Dec 18, 2015

2.3 is still maintained.

@stof
Copy link
Member

stof commented Dec 18, 2015

IMO, it makes sense to add tests in 2.3

@eventhorizonpl
Copy link
Author

Ok, I'll rebase it against 2.3

@eventhorizonpl
Copy link
Author

Here is a new PR #17069

Tobion added a commit that referenced this pull request Dec 24, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

improve BrowserKit test coverage p1

Hi,

This PR improves BrowserKit test coverage.

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| License       | MIT

I rebased previous PR #16549 to 2.3

Commits
-------

0261b48 improve BrowserKit test coverage p1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants