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

Skip to content

Commit ce7c8df

Browse files
committed
TST: correct expected canvas sizes
The values in the test were the buggy values. We expect a (5, 2) in figure at 120 dpi to be (600, 240) screen pixels.
1 parent cfff335 commit ce7c8df

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

lib/matplotlib/tests/test_backend_qt5.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -125,6 +125,7 @@ def test_dpi_ratio_change():
125125
size = qt_canvas.size()
126126

127127
qt_canvas.manager.show()
128+
qt_canvas.draw()
128129
qApp.processEvents()
129130

130131
# The DPI and the renderer width/height change
@@ -133,8 +134,8 @@ def test_dpi_ratio_change():
133134
assert qt_canvas.renderer.height == 720
134135

135136
# The actual widget size and figure physical size don't change
136-
assert size.width() == 200
137-
assert size.height() == 80
137+
assert size.width() == 600
138+
assert size.height() == 240
138139
assert_equal(qt_canvas.get_width_height(), (600, 240))
139140
assert_equal(fig.get_size_inches(), (5, 2))
140141

@@ -151,7 +152,7 @@ def test_dpi_ratio_change():
151152
assert qt_canvas.renderer.height == 480
152153

153154
# The actual widget size and figure physical size don't change
154-
assert size.width() == 200
155-
assert size.height() == 80
155+
assert size.width() == 600
156+
assert size.height() == 240
156157
assert_equal(qt_canvas.get_width_height(), (600, 240))
157158
assert_equal(fig.get_size_inches(), (5, 2))

0 commit comments

Comments
 (0)