Прошу помощи с ошибкой при запуске тестов для DiscourseRedis::Connector

Разработка
1

В течение последних нескольких дней при попытке запустить тесты локально я получаю эту ошибку:

$ bundle exec rake autospec

...

Произошла ошибка при загрузке ./spec/components/discourse_redis_spec.rb.
Failure/Error:
    describe DiscourseRedis::Connector do
      let(:connector) { DiscourseRedis::Connector.new(config) }
  
      after do
        fallback_handler.master = true
      end
  
      it 'should return the master config when master is up' do
        expect(connector.resolve).to eq(config)
      end

NameError:
  uninitialized constant DiscourseRedis::Connector
  Did you mean?  ConnectionPool

Я не знаю, что я мог случайно изменить, из-за чего моя система больше не может запускать тесты. Redis точно запущен, так что это не побочный эффект от неработающего Redis.

Эта ошибка возникает даже при запуске тестов на чистой копии ветки tests-passed, так что проблема в моей среде, но я не знаю, с чего начать поиск.

Есть какие-нибудь предложения?

2

Похоже, нам удалось добавить неработающие тесты. Я отключил их в этом коммите:

Однако у нас есть два последующих шага:

  1. Исправить неработающие тесты.
  2. Убедиться, что в будущем мы не сможем добавить их снова.
3

Спасибо, что посмотрели это! Неудачные спецификации попали в tests-passed из-за периодических проблем?

Я обновил код, и на ac9577bcc7b69593bcdd204368d35492fc0cee15 я всё ещё вижу точно такую же ошибку локально. Значит, вероятно, проблема не одна. :confused:

4

Извините, моя вина. Я забыл запушить :facepalm: Я обновил предыдущий пост с правильным коммитом.

И да, это означает, что сломанные тесты попали в tests-passed. К счастью, сам Discourse в порядке — нужно обновить тесты, а не исправлять ошибки в нашем приложении. Тем не менее, мы не допустим этого в будущем.

5

Всё уже было! Тесты теперь успешно выполняются.

Извини, что не спросил раньше. Я знаю, что я не в мейнстриме разработчиков Discourse, так как работаю на Fedora, а не на Ubuntu, поэтому у меня предвзятое мнение, будто на моей машине есть какая-то особенность окружения, которая просто отличается. Спасибо, что помогли мне запустить тестирование!

6

Я вижу, что есть разногласия по поводу отключения этих тестов из коммита 25db91e35180b2707ca4a9f7ffd173823f86f18d, который отменяет ваши изменения.

Но дело не в том, что за этим последовало исправление проблемы, потому что я тестировал на d775338d637d9ef3843acf227a138e8f11352151 (master), и на момент моего обновления тесты проходили, однако проблема вернулась.

По крайней мере, я добился прохождения тестов локально и выяснил, что нужно сделать иначе, чтобы учесть новые механизмы безопасности, которые я внедрил в скрипт миграции.

@tgxworld, какой план по разрешению конфликта? Я могу временно применить cherry-pick b1114b9a20343525c805d7339bd64954965a12f9 к основе моей ветки, когда мне нужно будет протестировать в это время?

7

Тесты должны были быть удалены, но я не заметил этого, потому что в нашей CI была ошибка: она не падала при возникновении ошибки. Это исправлено в