mirror of
				https://github.com/python/cpython.git
				synced 2025-11-03 19:34:08 +00:00 
			
		
		
		
	Issue #20452: add more info in case of test_asyncio failure to try to debug the
failure on buildbot "x86 Ubuntu Shared 3.x"
This commit is contained in:
		
							parent
							
								
									a849be9c64
								
							
						
					
					
						commit
						0278032110
					
				
					 2 changed files with 19 additions and 2 deletions
				
			
		| 
						 | 
					@ -620,10 +620,18 @@ class BaseEventLoop(events.AbstractEventLoop):
 | 
				
			||||||
                timeout = min(timeout, deadline)
 | 
					                timeout = min(timeout, deadline)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        # TODO: Instrumentation only in debug mode?
 | 
					        # TODO: Instrumentation only in debug mode?
 | 
				
			||||||
        if logger.isEnabledFor(logging.INFO):
 | 
					        # FIXME: don't force log (issue #20452)
 | 
				
			||||||
 | 
					        if True: #logger.isEnabledFor(logging.INFO):
 | 
				
			||||||
            t0 = self.time()
 | 
					            t0 = self.time()
 | 
				
			||||||
            event_list = self._selector.select(timeout)
 | 
					            event_list = self._selector.select(timeout)
 | 
				
			||||||
            t1 = self.time()
 | 
					            t1 = self.time()
 | 
				
			||||||
 | 
					            # FIXME: remove these debug info (issue #20452)
 | 
				
			||||||
 | 
					            dt = t1-t0
 | 
				
			||||||
 | 
					            if dt < timeout and not event_list:
 | 
				
			||||||
 | 
					                print("WARNING: selector.select(timeout=%.20f) took dt=%.20f sec (dt-timeout=%+.20f)"
 | 
				
			||||||
 | 
					                      % (timeout, dt, dt-timeout), file=sys.__stdout__)
 | 
				
			||||||
 | 
					                print("WARNING: dt+%.20f > timeout? %s"
 | 
				
			||||||
 | 
					                      % (self._granularity, (dt + self._granularity) > timeout), file=sys.__stdout__)
 | 
				
			||||||
            if t1-t0 >= 1:
 | 
					            if t1-t0 >= 1:
 | 
				
			||||||
                level = logging.INFO
 | 
					                level = logging.INFO
 | 
				
			||||||
            else:
 | 
					            else:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1157,6 +1157,10 @@ class EventLoopTestsMixin:
 | 
				
			||||||
        w.close()
 | 
					        w.close()
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    def test_timeout_rounding(self):
 | 
					    def test_timeout_rounding(self):
 | 
				
			||||||
 | 
					        # FIXME: remove this imports, used for debug purpose (issue #20452)
 | 
				
			||||||
 | 
					        import time
 | 
				
			||||||
 | 
					        import platform
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        def _run_once():
 | 
					        def _run_once():
 | 
				
			||||||
            self.loop._run_once_counter += 1
 | 
					            self.loop._run_once_counter += 1
 | 
				
			||||||
            orig_run_once()
 | 
					            orig_run_once()
 | 
				
			||||||
| 
						 | 
					@ -1177,7 +1181,12 @@ class EventLoopTestsMixin:
 | 
				
			||||||
 | 
					
 | 
				
			||||||
        self.loop.run_until_complete(wait())
 | 
					        self.loop.run_until_complete(wait())
 | 
				
			||||||
        calls.append(self.loop._run_once_counter)
 | 
					        calls.append(self.loop._run_once_counter)
 | 
				
			||||||
        self.assertEqual(calls, [1, 3, 5, 6])
 | 
					        self.assertEqual(calls, [1, 3, 5, 6],
 | 
				
			||||||
 | 
					                         # FIXME: remove these info, used for debug purpose (issue #20452)
 | 
				
			||||||
 | 
					                         (self.loop._granularity,
 | 
				
			||||||
 | 
					                          self.loop._selector.resolution,
 | 
				
			||||||
 | 
					                          time.get_clock_info('monotonic'),
 | 
				
			||||||
 | 
					                          platform.platform()))
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					
 | 
				
			||||||
class SubprocessTestsMixin:
 | 
					class SubprocessTestsMixin:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
	Add table
		Add a link
		
	
		Reference in a new issue