New issue
Advanced search Search tips

Issue 781723 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Task



Sign in to add a comment

Add tests for Tricium RPCs

Project Member Reported by emso@chromium.org, Nov 6 2017

Issue description

Improve test coverage by adding tests for Tricium RPCs, both for the public API and for the admin APIs.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Feb 7 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/c78b44136f8049aa59e60e0d009590a83fd24afd

commit c78b44136f8049aa59e60e0d009590a83fd24afd
Author: Quinten Yearsley <qyearsley@chromium.org>
Date: Wed Feb 07 22:07:56 2018

Add unit test for Progress RPC function

Bug: 781723
Change-Id: I87ac1fb478ed67d322f4459cfabe48cf3dab9510
Reviewed-on: https://chromium-review.googlesource.com/907469
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>

[modify] https://crrev.com/c78b44136f8049aa59e60e0d009590a83fd24afd/go/src/infra/tricium/appengine/frontend/rpc_progress_test.go
[modify] https://crrev.com/c78b44136f8049aa59e60e0d009590a83fd24afd/go/src/infra/tricium/appengine/frontend/frontend.infra_testing

Owner: qyears...@chromium.org
Status: Assigned (was: Available)
Tests that already have public RPC function tests:

 frontend/rpc_progress_test.go
 frontend/rpc_report_test.go

Tests which might be expanded/improved:

 frontend/rpc_project_progress_test.go

 frontend/rpc_feedback_test.go
 frontend/rpc_results_test.go
 frontend/rpc_analyze_test.go
 tracker/rpc_workflow_launched_test.go
 tracker/rpc_worker_launched_test.go
 tracker/rpc_worker_done_test.go
 driver/rpc_collect_test.go
 driver/rpc_trigger_test.go
 launcher/rpc_launch_test.go
 gerrit/rpc_report_results_test.go

Labels: Hotlist-CodeHealth
Owner: ----
Status: Available (was: Assigned)
Components: Infra>Platform>Tricium
Components: -Infra>CodeAnalysis
Labels: -Tricium
Labels: Hotlist-GoodFirstBug

Sign in to add a comment