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

Complete tests for QgsMapToolZoom #20450

Closed
qgib opened this issue Feb 25, 2015 · 3 comments
Closed

Complete tests for QgsMapToolZoom #20450

qgib opened this issue Feb 25, 2015 · 3 comments
Labels
Chore GitHub and other CI infrastructure changes Feature Request

Comments

@qgib
Copy link
Contributor

qgib commented Feb 25, 2015

Author Name: Sandro Santilli (@strk)
Original Redmine Issue: 12265

Redmine category:tests_suite


The class behavior is not fully tested. A single test for a very specific case is available a the time of writing in
https://github.com/qgis/QGIS/blob/master/tests/src/gui/testqgsmaptoolzoom.cpp

Absence of tests result in an higher risk of regressions being introduced, as happened in 2.8 (#20427)

@qgib
Copy link
Contributor Author

qgib commented May 8, 2015

Author Name: Sandro Santilli (@strk)


  • tracker_id was changed from 1 to 2

@qgib
Copy link
Contributor Author

qgib commented May 8, 2015

Author Name: Sandro Santilli (@strk)


  • fixed_version_id was changed from Version 2.8.1 to Future Release - Nice to have

@qgib
Copy link
Contributor Author

qgib commented May 30, 2016

Author Name: Nyall Dawson (@nyalldawson)


While I'd also like to see more tests here, I don't think this warrants a bug report. Otherwise we'd be opening these for 100's of classes across the codebase.


  • resolution was changed from to invalid
  • status_id was changed from Open to Closed

@qgib qgib added Feature Request Chore GitHub and other CI infrastructure changes labels May 25, 2019
@qgib qgib added this to the Future Release - Nice to have milestone May 25, 2019
@qgib qgib closed this as completed May 25, 2019
@qgib qgib mentioned this issue May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore GitHub and other CI infrastructure changes Feature Request
Projects
None yet
Development

No branches or pull requests

1 participant