Yair Zaslavsky has posted comments on this change.

Change subject: frontend:  node upgrade message improvement
......................................................................


Patch Set 7:

Douglas - the following unit test demonstrates that your last check should have 
not produced an upgrade message -

@Test
    public void testShouldAlertUpgrade() {
        RpmVersion rpm = new 
RpmVersion("ovirt-node-iso-2.6.1-20130212.fc18.iso", "ovirt-node-iso", true);
        ArrayList<RpmVersion> rpmList = new ArrayList<RpmVersion>();
        rpmList.add(rpm);
        String[] hostOs = { "oVirt Node", "2.6.1", "20130212.fc18" };
        assertFalse(shouldAlertUpgrade(rpmList, hostOs));
    }

-- 
To view, visit http://gerrit.ovirt.org/17987
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I76bd2948524110ec80923e855bd1f49e09e17046
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Barak Azulay <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to