Не совсем корректно. Этот тест не падает.
Падает тест, который проверяет `self.assertEquals(0, 1)` - он бросает ошибку `AssertionError`, которая перехватывается и принимается решение что тест упал, __НО__ в данном случае, целевая функция бросает исключение - происходит ошибка. Исключение поднимается выше и поскольку исключение было `AssertionError` - принимается решение что тест упал.
Делать надо хорошо. А именно:
Переписывается функция, так чтобы она бросала исключение `ValueError` когда значение недопустимое, а в этом тесте декоратор убирается и тело переписывается как `self.assertRaises(ValueError, number_to_weekday, 0)`
Таким образом мы уверены что поднимается именно то исключение, которое задумывалось(не произошло еще какой-нить ошибки). И __поэтому__ не надо использовать в коде выражения assert.