Commit 963a5f8c authored by Canek Peláez's avatar Canek Peláez
Browse files

Fix track/total bug and add unit test.

parent 0a3e9a46

lib/mlm/file-tags.vala

+1 −1
Original line number Diff line number Diff line
@@ -473,7 +473,7 @@ namespace MLM { 
            if (new_total > 0 && new_track < 0)

                new_total = -1;

            string value = (new_track == -1) ? null :

                "%d/%d".printf(new_total, new_track);

                "%d/%d".printf(new_track, new_total);

            define_text_value(FrameId.TRACK, value);

            if (string_frames.has_key(FrameId.TRACK)) {

                _track = new_track;

test/test-file-tags.vala

+12 −0
Original line number Diff line number Diff line
@@ -270,6 +270,18 @@ namespace MLM.Test { 
            GLib.assert(tags.track == 7);

            GLib.assert(tags.total == 7);

            tags = null;

            tags = new FileTags(path);

            tags.track = -1;

            tags.update();

            GLib.assert(tags.track == -1);

            GLib.assert(tags.total == -1);

            tags = null;

            tags = new FileTags(path);

            tags.track = 3;

            tags.total = 7;

            tags.update();

            GLib.assert(tags.track == 3);

            GLib.assert(tags.total == 7);

        }



        /**