OLD | NEW |
---|---|
1 # Copyright 2016 The Chromium Authors. All rights reserved. | 1 # Copyright 2016 The Chromium Authors. All rights reserved. |
2 # Use of this source code is governed by a BSD-style license that can be | 2 # Use of this source code is governed by a BSD-style license that can be |
3 # found in the LICENSE file. | 3 # found in the LICENSE file. |
4 | 4 |
5 import contextlib | 5 import contextlib |
6 | 6 |
7 from telemetry import decorators | 7 from telemetry import decorators |
8 from telemetry import page as page_module | 8 from telemetry import page as page_module |
9 from telemetry import story | 9 from telemetry import story |
10 from telemetry.page import cache_temperature | 10 from telemetry.page import cache_temperature |
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
43 def testEnsureAny(self): | 43 def testEnsureAny(self): |
44 with self.captureTrace(): | 44 with self.captureTrace(): |
45 story_set = story.StorySet() | 45 story_set = story.StorySet() |
46 page = page_module.Page( | 46 page = page_module.Page( |
47 'http://google.com', page_set=story_set, | 47 'http://google.com', page_set=story_set, |
48 cache_temperature=cache_temperature.ANY, name='http://google.com') | 48 cache_temperature=cache_temperature.ANY, name='http://google.com') |
49 cache_temperature.EnsurePageCacheTemperature(page, self._browser) | 49 cache_temperature.EnsurePageCacheTemperature(page, self._browser) |
50 | 50 |
51 markers = self.traceMarkers() | 51 markers = self.traceMarkers() |
52 self.assertNotIn('telemetry.internal.ensure_diskcache.start', markers) | 52 self.assertNotIn('telemetry.internal.ensure_diskcache.start', markers) |
53 self.assertNotIn('telemetry.internal.warm_cache.start', markers) | 53 self.assertTrue( |
54 all('telemetry.internal.warm_cache' not in maker for maker in markers)) | |
kouhei (in TOK)
2017/09/26 04:40:52
s/maker/marker/
why not assertNotIn?
yukiy
2017/09/26 05:23:32
Because I added temperature for identifier in cach
| |
54 | 55 |
55 @decorators.Enabled('has tabs') | 56 @decorators.Enabled('has tabs') |
56 @decorators.Disabled('chromeos') | 57 @decorators.Disabled('chromeos') |
57 def testEnsureCold(self): | 58 def testEnsureCold(self): |
58 with self.captureTrace(): | 59 with self.captureTrace(): |
59 story_set = story.StorySet() | 60 story_set = story.StorySet() |
60 page = page_module.Page( | 61 page = page_module.Page( |
61 'http://google.com', page_set=story_set, | 62 'http://google.com', page_set=story_set, |
62 cache_temperature=cache_temperature.COLD, name='http://google.com') | 63 cache_temperature=cache_temperature.COLD, name='http://google.com') |
63 cache_temperature.EnsurePageCacheTemperature(page, self._browser) | 64 cache_temperature.EnsurePageCacheTemperature(page, self._browser) |
(...skipping 12 matching lines...) Expand all Loading... | |
76 cache_temperature.EnsurePageCacheTemperature(page, self._browser) | 77 cache_temperature.EnsurePageCacheTemperature(page, self._browser) |
77 | 78 |
78 previous_page = page | 79 previous_page = page |
79 page = page_module.Page( | 80 page = page_module.Page( |
80 'http://google.com', page_set=story_set, | 81 'http://google.com', page_set=story_set, |
81 cache_temperature=cache_temperature.WARM, name='http://google.com') | 82 cache_temperature=cache_temperature.WARM, name='http://google.com') |
82 cache_temperature.EnsurePageCacheTemperature( | 83 cache_temperature.EnsurePageCacheTemperature( |
83 page, self._browser, previous_page) | 84 page, self._browser, previous_page) |
84 | 85 |
85 markers = self.traceMarkers() | 86 markers = self.traceMarkers() |
86 self.assertNotIn('telemetry.internal.warm_cache.start', markers) | 87 self.assertTrue( |
88 all('telemetry.internal.warm_cache' not in maker for maker in markers)) | |
kouhei (in TOK)
2017/09/26 04:40:52
ditto
| |
87 | 89 |
88 @decorators.Enabled('has tabs') | 90 @decorators.Enabled('has tabs') |
89 @decorators.Disabled('chromeos') | 91 @decorators.Disabled('chromeos') |
90 def testEnsureWarmFromScratch(self): | 92 def testEnsureWarmFromScratch(self): |
91 with self.captureTrace(): | 93 with self.captureTrace(): |
92 story_set = story.StorySet() | 94 story_set = story.StorySet() |
93 page = page_module.Page( | 95 page = page_module.Page( |
94 'http://google.com', page_set=story_set, | 96 'http://google.com', page_set=story_set, |
95 cache_temperature=cache_temperature.WARM, name='http://google.com') | 97 cache_temperature=cache_temperature.WARM, name='http://google.com') |
96 cache_temperature.EnsurePageCacheTemperature(page, self._browser) | 98 cache_temperature.EnsurePageCacheTemperature(page, self._browser) |
97 | 99 |
98 markers = self.traceMarkers() | 100 markers = self.traceMarkers() |
99 self.assertIn('telemetry.internal.warm_cache.start', markers) | 101 self.assertIn('telemetry.internal.warm_cache.warm.start', markers) |
100 self.assertIn('telemetry.internal.warm_cache.end', markers) | 102 self.assertIn('telemetry.internal.warm_cache.warm.end', markers) |
103 | |
104 @decorators.Enabled('has tabs') | |
105 def testEnsureHotAfterColdAndWarmRun(self): | |
106 with self.captureTrace(): | |
107 story_set = story.StorySet() | |
108 page = page_module.Page( | |
109 'http://google.com', page_set=story_set, | |
110 cache_temperature=cache_temperature.COLD, name='http://google.com') | |
111 cache_temperature.EnsurePageCacheTemperature(page, self._browser) | |
112 | |
113 previous_page = page | |
114 page = page_module.Page( | |
115 'http://google.com', page_set=story_set, | |
116 cache_temperature=cache_temperature.WARM, name='http://google.com') | |
117 cache_temperature.EnsurePageCacheTemperature( | |
118 page, self._browser, previous_page) | |
119 | |
120 previous_page = page | |
121 page = page_module.Page( | |
122 'http://google.com', page_set=story_set, | |
123 cache_temperature=cache_temperature.HOT, name='http://google.com') | |
124 cache_temperature.EnsurePageCacheTemperature( | |
125 page, self._browser, previous_page) | |
126 | |
127 markers = self.traceMarkers() | |
128 self.assertTrue( | |
129 all('telemetry.internal.warm_cache' not in maker for maker in markers)) | |
kouhei (in TOK)
2017/09/26 04:40:52
ditto
| |
130 | |
131 def testEnsureHotAfterColdRun(self): | |
132 with self.captureTrace(): | |
133 story_set = story.StorySet() | |
134 page = page_module.Page( | |
135 'http://google.com', page_set=story_set, | |
136 cache_temperature=cache_temperature.COLD, name='http://google.com') | |
137 cache_temperature.EnsurePageCacheTemperature(page, self._browser) | |
138 | |
139 previous_page = page | |
140 page = page_module.Page( | |
141 'http://google.com', page_set=story_set, | |
142 cache_temperature=cache_temperature.HOT, name='http://google.com') | |
143 cache_temperature.EnsurePageCacheTemperature( | |
144 page, self._browser, previous_page) | |
145 | |
146 markers = self.traceMarkers() | |
147 # After navigation for another origin url, traces in previous origin page | |
148 # does not appear in |makers|, so we can not check this: | |
149 # self.assertIn('telemetry.internal.warm_cache.hot.start', markers) | |
150 # TODO: Ensure all traces are in |makers| | |
151 self.assertIn('telemetry.internal.warm_cache.hot.end', markers) | |
152 | |
153 @decorators.Enabled('has tabs') | |
154 @decorators.Disabled('chromeos') | |
155 def testEnsureHotFromScratch(self): | |
156 with self.captureTrace(): | |
157 story_set = story.StorySet() | |
158 page = page_module.Page( | |
159 'http://google.com', page_set=story_set, | |
160 cache_temperature=cache_temperature.HOT, name='http://google.com') | |
161 cache_temperature.EnsurePageCacheTemperature(page, self._browser) | |
162 | |
163 markers = self.traceMarkers() | |
164 self.assertIn('telemetry.internal.warm_cache.warm.start', markers) | |
165 self.assertIn('telemetry.internal.warm_cache.warm.end', markers) | |
166 self.assertIn('telemetry.internal.warm_cache.hot.start', markers) | |
167 self.assertIn('telemetry.internal.warm_cache.hot.end', markers) | |
OLD | NEW |