November 26th, 2015

404

турецкий питон

- elif isinstance(other, str) or other is None:
+ elif isinstance(other, basestring) or other is None:

Я знаю, что в третьем пофиксили. Всё равно отвратительно.

(Фикшу адский баг...)
404

coverage как источник неудобных вопросов

Понятно, что любая строкомерка порочна. Но иногда полезна.

Код (уже из фикса):
-        try:
-            os.rename(filename, filename + '.backup')
-        except OSError, IOError:
-            pass
+        os.rename(filename, filename + '.backup')

В функции save предусмотрено делать бэкапы. coverage говорит, что ни один тест не проверяет except ветку. Пытаюсь написать. Пытаюсь понять, "а что должно быть-то?", и понимаю, что разумного ответа на вопрос нет. Не могу сделать бэкап... Просто перезатереть? Плохо. Сказать пользователю? Но это дебри классов и тут ничего "сказать" нельзя.

Почесал в затылке и решил, что лучше пусть адекватно фейлится, в текущем виде мешая сохранению.