дыбр

Sep. 3rd, 2020 07:23 pm
kdanilov: Кирша Данилов (Default)
[personal profile] kdanilov
Ну это, т.к. с утра в госпиталь ездил, на кардиографию, то еле успел до часу зафайлить "новую" багу, про кешинг, и присоединить мой пулреквест к ней. Очень важная деятельность в изготовлении софта.

Но в час митинга не было, т.к. мы постепенно переходим на общеконторский режим дня, или, скорее, недели, а там по четвергам никаких митингов. Это все спасибо Марисе, она в Гугле завела этот no-meeting Thursday, и пошло по нашему уезду.

На квипе идет как бы чуть ли не борьба оптимизаций. Мой простой кеш, и сложная хрень от Аарона. Ну это ладно, я ему пишу, хорошо, мол, твоя вещь хорошая, давай сотрудничать; а он такой - не, вообще кеш не нужен (потому-то и потому-то), а я такой - чо, весь мир пользуется, в компе полно всякого кеша, а нам вот как раз не годится? В конторе это рекомендуемое решение; а твое тоже подтвеждается какими-то конторскими документами? Причем, мне абсолютно пох, абсолютно, я могу помогать ему его херню имплементировать (только у него тестов ноль, а перформанс-тестов - минус десять). Мне уже никто не пеняет, что у меня перформанс-тестов нету, у меня есть.

Ну и т.д. Каролин на мой док понаписала кучу вопросов; очень хорошо. Она такая ученая бюрократка. Я ответил на все эти вопросы. Смысла никакого нет, потому что ответы все равно никто не читает. Но важно, что ответил. Ну в смысле, кому важно? Начальству. Птичку поставят. Мне-то пох.

Шибин зато на мой пулреквест (с кешем) понаписал массу очень ценных каментов, так что я все это пофиксил там. Спасибо ему большое.

Тут кое-кто может спросить, выражаю ли я спасибо в письменном виде. Поверьте, больше, чем здесь упоминаю. И уверен, что моя улыбка в слаке совершенно не видна. В слаке вообще не надо улыбаться. Сурьезная рожа должна быть. Мы работаем.

Ну а так, ну что, ну. Конечно, я прабовский код должен был потестировать на своей херне - но у меня и тест-плана нету, ну. Теоретически я знаю, что надо делать, но практически лень.

Но мне кажется, что я сачкую как японец. Т.е. совесть немножко гложет, но если со стороны, то я какой-то гиперактивный ебанат, все время какой-то код пытаюсь коммитить, а Кевин такой - а это зачем, это к какому кейсу? Ну я могу зафайлить кейс "давайте улучшим наши тесты", но ведь не поймут-с. Буду как поручик Ржевский, евпочя.

Ответы никто не читает?

Date: 2020-09-04 03:27 am (UTC)
dennisgorelik: 2020-06-13 in my home office (Default)
From: [personal profile] dennisgorelik
> Смысла никакого нет, потому что ответы все равно никто не читает.

Откуда ты знаешь, что ответы никто не читает?
А Каролин - тоже не читает ответы на вопросы, которые она задала?

Гиперактивность

Date: 2020-09-04 03:31 am (UTC)
dennisgorelik: 2020-06-13 in my home office (Default)
From: [personal profile] dennisgorelik
> Кевин такой - а это зачем, это к какому кейсу?

Семь раз отмерь - один раз отрежь.

Все, что ты коммитишь - нужно же ревьюить и тестировать. Поэтому Кевин с осторожностью относится к новому коду.

Re: Гиперактивность

Date: 2020-09-04 07:24 am (UTC)
dennisgorelik: 2020-06-13 in my home office (Default)
From: [personal profile] dennisgorelik
> Ты понимаешь, что речь идет о чистке тестов?

Сейчас, когда ты об этом явно написал - понимаю.

> Как их еще тестировать, кроме как гонять?

Можно еще читать тесты и пытаться найти ошибки.
Так, обычно, дольше, конечно.

> выкидываю бессмысленные System.assert(true, 'this test passed');.

Ты уменьшаешь количества кода в тестах?
А Кевин тебя спрашивает, зачем нужно уменьшать код в тестах?

Re: Гиперактивность

Date: 2020-09-04 08:13 pm (UTC)
dennisgorelik: 2020-06-13 in my home office (Default)
From: [personal profile] dennisgorelik
> Не, у меня объяснение есть - я просто вырезал это из большого старого пиара.

Кевин согласился с этим объяснением?

> Тесты у нас во многом фейковые; вот это и напрягает.

Даже фэйковый тест, обычно, тестирует хотя бы успешное завершение исполнения по одному из сценариев.

Profile

kdanilov: Кирша Данилов (Default)
kdanilov

November 2022

S M T W T F S
  12345
6789101112
13141516171819
20212223242526
2728 2930   

Style Credit

Expand Cut Tags

No cut tags
Page generated Aug. 30th, 2025 10:20 am
Powered by Dreamwidth Studios