webkit-patch rebaseline-expectations doesn't work w/o failures specified
Created attachment 167807 [details] Patch
Comment on attachment 167807 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167807&action=review > Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py:872 > + return self._model.get_test_set(REBASELINE, IMAGE) | self._model.get_test_set(REBASELINE, FAIL) | self._model.get_test_set(REBASELINE, PASS) Can you just use self._model.get_test_set("REBASELINE")? In addition to the fixme to get rid of get_test_set, I worry that we'll eventually do the cleanup to not mark skip/rebaseline tests as pass, but just have the expectation be skip/rebaseline. Then this code will break again.
(In reply to comment #2) > (From update of attachment 167807 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=167807&action=review > > > Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py:872 > > + return self._model.get_test_set(REBASELINE, IMAGE) | self._model.get_test_set(REBASELINE, FAIL) | self._model.get_test_set(REBASELINE, PASS) > > Can you just use self._model.get_test_set("REBASELINE")? > Oh, yeah, I guess you can. I was thinking (a) that you had to have expectations, and (b) that we wouldn't want to rebaseline something if it was marked Crash or Timeout, but perhaps that's overly paranoid. > In addition to the fixme to get rid of get_test_set, I worry that we'll eventually do the cleanup to not mark skip/rebaseline tests as pass, but just have the expectation be skip/rebaseline. Then this code will break again. But the test should catch it if it breaks again, right?
Comment on attachment 167807 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=167807&action=review >>> Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py:872 >>> + return self._model.get_test_set(REBASELINE, IMAGE) | self._model.get_test_set(REBASELINE, FAIL) | self._model.get_test_set(REBASELINE, PASS) >> >> Can you just use self._model.get_test_set("REBASELINE")? >> >> In addition to the fixme to get rid of get_test_set, I worry that we'll eventually do the cleanup to not mark skip/rebaseline tests as pass, but just have the expectation be skip/rebaseline. Then this code will break again. > > Oh, yeah, I guess you can. I was thinking (a) that you had to have expectations, and (b) that we wouldn't want to rebaseline something if it was marked Crash or Timeout, but perhaps that's overly paranoid. Yeah, I think that's overly paranoid. If you add rebaseline, you're asking for it I think.
Created attachment 167845 [details] just get all of the tests marked for rebaselining
Comment on attachment 167845 [details] just get all of the tests marked for rebaselining Clearing flags on attachment: 167845 Committed r130819: <http://trac.webkit.org/changeset/130819>
All reviewed patches have been landed. Closing bug.