Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(105)

Side by Side Diff: appengine/findit/crash/test/findit_for_crash_test.py

Issue 1950123003: [Findit] Fetch DEPS from buildspec/ instead of trunk for chrome official builds. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Update doc string. Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
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 from collections import defaultdict 5 from collections import defaultdict
6 6
7 from common.blame import Region, Blame 7 from common.blame import Region, Blame
8 from common.change_log import ChangeLog 8 from common.change_log import ChangeLog
9 from common.dependency import Dependency, DependencyRoll 9 from common.dependency import Dependency, DependencyRoll
10 from common.git_repository import GitRepository 10 from common.git_repository import GitRepository
(...skipping 338 matching lines...) Expand 10 before | Expand all | Expand 10 after
349 expected_match_results = [ 349 expected_match_results = [
350 { 350 {
351 'url': 'https://repo.test/+/1', 351 'url': 'https://repo.test/+/1',
352 'revision': '1', 352 'revision': '1',
353 'dep_path': 'src/', 353 'dep_path': 'src/',
354 'component': '', 354 'component': '',
355 'author': 'r@chromium.org', 355 'author': 'r@chromium.org',
356 'time': 'Thu Mar 31 21:24:43 2016', 356 'time': 'Thu Mar 31 21:24:43 2016',
357 'reason': ('1. Top frame changed is frame #0 (score: 1)\n' 357 'reason': ('1. Top frame changed is frame #0 (score: 1)\n'
358 '2. Minimum distance to crashed line is 0 (score: 1)\n' 358 '2. Minimum distance to crashed line is 0 (score: 1)\n'
359 '\nChanged file a.cc which crashed in func (#0)' 359 '\nChanged file a.cc crashed in func (#0)'
360 ', func (#1)'), 360 ', func (#1)'),
361 'confidence': 1, 361 'confidence': 1,
362 }, 362 },
363 { 363 {
364 'url': 'https://repo.test/+/3', 364 'url': 'https://repo.test/+/3',
365 'revision': '3', 365 'revision': '3',
366 'dep_path': 'src/', 366 'dep_path': 'src/',
367 'component': '', 367 'component': '',
368 'author': 'e@chromium.org', 368 'author': 'e@chromium.org',
369 'time': 'Thu Apr 1 21:24:43 2016', 369 'time': 'Thu Apr 1 21:24:43 2016',
370 'reason': ('1. Top frame changed is frame #5 (score: 0)\n' 370 'reason': ('1. Top frame changed is frame #5 (score: 0)\n'
371 '2. Minimum distance to crashed line is 20 (score: 0)\n' 371 '2. Minimum distance to crashed line is 20 (score: 0)\n'
372 '\nChanged file f.cc which crashed in func (#5)'), 372 '\nChanged file f.cc crashed in func (#5)'),
373 'confidence': 0.22857142857142856, 373 'confidence': 0.22857142857142856,
374 }, 374 },
375 ] 375 ]
376 376
377 regression_deps_rolls = {'src/': DependencyRoll('src/', 'https://repo', 377 regression_deps_rolls = {'src/': DependencyRoll('src/', 'https://repo',
378 '1', '2')} 378 '1', '2')}
379 379
380 self.assertEqual(findit_for_crash.FindItForCrash( 380 self.assertEqual(findit_for_crash.FindItForCrash(
381 Stacktrace(), regression_deps_rolls, {}), expected_match_results) 381 Stacktrace(), regression_deps_rolls, {}), expected_match_results)
OLDNEW
« no previous file with comments | « appengine/findit/crash/test/callstack_filters_test.py ('k') | appengine/findit/crash/test/fracas_crash_pipeline_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698