Project

General

Profile

Bug #11845

acquire-spray test could be improved

Added by John Levon 3 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:

Description

acquire-spray doesn't need the lockstat invocation for unit testing, and it's a littly racy WRT the monitor

History

#1

Updated by John Levon 3 months ago

Tested it by hand, as well as making sure an os-tests run skips this check, but runs kmc-update, in a loop

#2

Updated by Electric Monk 3 months ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

git commit 7ca0d613a17085b59a8b231daa9eddfa74fd83ee

commit  7ca0d613a17085b59a8b231daa9eddfa74fd83ee
Author: John Levon <john.levon@joyent.com>
Date:   2019-10-29T14:43:26.000Z

    11845 acquire-spray test could be improved
    11857 kmc-update test missing from runfile
    Reviewed by: Robert Mustacchi <rm@fingolfin.org>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF