Bug 25501 - Удалены тесты, прошу вернуть
Summary: Удалены тесты, прошу вернуть
Status: RESOLVED LATER
Alias: None
Product: Sisyphus
Classification: Development
Component: ruby (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: timonbl4@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-22 22:49 MSK by AEN
Modified: 2011-04-23 11:43 MSK (History)
9 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Comment 2 Anton Farygin 2011-04-22 23:03:55 MSK
Алексей, по твоей ссылке тесты как раз вернули, а не удалили. Смотри на патч внимательнее, пожалуйста.

Конечно тесты будут возвращать по мере исправления.
Comment 3 Anton Farygin 2011-04-22 23:13:49 MSK
Тимур, посмотри пожалуйста, почему именно не проходят тесты, нужно ли их исправлять и каким именно образом.

Т.к. это ruby 1.9, то приоритет соответственно низкий, после исправления сборки модулей.
Comment 4 Andriy Stepanov (stanv) 2011-04-23 01:19:37 MSK
(В ответ на комментарий №1)
> См.
> http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=summary

1.
Как делать нельзя ?

2.
Традицию удалять не проходящие тесты ввел "кто-то" до меня.

3.
>> См.
>> http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=summary
Куда смотреть?

4.
Я удалил по сути четыре теста.

5.
Прежде чем удалят я пытался выяснить суть дела у icesik@ && raorn@....
Чем история закончилась - можно догадаться.

6.
# XXX@stanv, next test doesn't passes.
 rm -rf test/test_pty.rb
 rm -rf test/drb/ut_array_drbssl.rb
 rm -rf test/drb/test_drbssl.rb
 rm -rf test/net/http/utils.rb
 rm -rf test/net/http/test_https.rb

Один их тестов относится к терминалам в hasher.

Последние два теста виснут до бесконечности - поскольку я в хешере росшарил сеть.

7.
По какой инициативе была открыта эта ошибка и где сама ошибка ?

Дайте пожалуйста ответы на мои вопросы.
Comment 5 Andriy Stepanov (stanv) 2011-04-23 01:28:52 MSK
Я удалил только три теста, несовместимые с хешер.
Все остальные тесты остались.
Т.е. 99% тестов выполняются.

1.
Почему это считается ошибкой?

2.
В руби 1.9.2 есть намного более глобальные проблемы.
Например: некорректная работа в много-поточной среде.


Эта ошибка все еще актуальна?
Comment 6 AEN 2011-04-23 01:41:21 MSK
(В ответ на комментарий №4)
> (В ответ на комментарий №1)
> > См.
> > http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=summary
> 
> 1.
> Как делать нельзя ?

Удалять тесты.

> 
> 2.
> Традицию удалять не проходящие тесты ввел "кто-то" до меня.
> 

Есть ссылка на коммит?

> 3.
> >> См.
> >> http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=summary
> Куда смотреть?
> 
> 4.
> Я удалил по сути четыре теста.
> 
> 5.
> Прежде чем удалят я пытался выяснить суть дела у icesik@ && raorn@....
> Чем история закончилась - можно догадаться.
> 
> 6.
> # XXX@stanv, next test doesn't passes.
>  rm -rf test/test_pty.rb
>  rm -rf test/drb/ut_array_drbssl.rb
>  rm -rf test/drb/test_drbssl.rb
>  rm -rf test/net/http/utils.rb
>  rm -rf test/net/http/test_https.rb
> 
> Один их тестов относится к терминалам в hasher.
> 
> Последние два теста виснут до бесконечности - поскольку я в хешере росшарил
> сеть.
> 
> 7.
> По какой инициативе была открыта эта ошибка и где сама ошибка ?

Исключение тестов -- ошибка. Тем более без сообщения и обоснования в changelog. Я понял Вас, спасибо. Но хочу разобраться с тем, кто все же начал удаление тестов.

> 
> Дайте пожалуйста ответы на мои вопросы.
Comment 7 AEN 2011-04-23 01:42:16 MSK
(В ответ на комментарий №5)
> Я удалил только три теста, несовместимые с хешер.
> Все остальные тесты остались.
> Т.е. 99% тестов выполняются.
> 
> 1.
> Почему это считается ошибкой?
> 
> 2.
> В руби 1.9.2 есть намного более глобальные проблемы.
> Например: некорректная работа в много-поточной среде.

Вешайте багу, пожалуйста.
Comment 8 Andriy Stepanov (stanv) 2011-04-23 10:03:21 MSK
(В ответ на комментарий №6)
> (В ответ на комментарий №4)
> > (В ответ на комментарий №1)
> > > См.
> > > http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=summary
> > 
> > 1.
> > Как делать нельзя ?
> 
> Удалять тесты.
>

Еще раз.
Тесты - есть и выполняются.
Я удалил только 5ть файлов, которые не работают в хешер.
Раз есть сомнения что я не правильно сделал, давайте разбираться.
Какой из этих файлов следует мне вернуть?

> > 
> > 2.
> > Традицию удалять не проходящие тесты ввел "кто-то" до меня.
> > 
> 
> Есть ссылка на коммит?

http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=blob;f=ruby.spec;h=7a3590cf621b730609934d1242d5c2d04bc6d489;hb=fc6177d8eac5f22d29a29c24e4893ac3714cbb8d


Это SPEC из руби, который был до меня.
Там тоже удаляются некоторые тесты.
Строка 301.


> 
> > 3.
> > >> См.
> > >> http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=summary
> > Куда смотреть?
> > 
> > 4.
> > Я удалил по сути четыре теста.
> > 
> > 5.
> > Прежде чем удалят я пытался выяснить суть дела у icesik@ && raorn@....
> > Чем история закончилась - можно догадаться.
> > 
> > 6.
> > # XXX@stanv, next test doesn't passes.
> >  rm -rf test/test_pty.rb
> >  rm -rf test/drb/ut_array_drbssl.rb
> >  rm -rf test/drb/test_drbssl.rb
> >  rm -rf test/net/http/utils.rb
> >  rm -rf test/net/http/test_https.rb
> > 
> > Один их тестов относится к терминалам в hasher.
> > 
> > Последние два теста виснут до бесконечности - поскольку я в хешере росшарил
> > сеть.
> > 
> > 7.
> > По какой инициативе была открыта эта ошибка и где сама ошибка ?
> 
> Исключение тестов -- ошибка. Тем более без сообщения и обоснования в changelog.
> Я понял Вас, спасибо. Но хочу разобраться с тем, кто все же начал удаление
> тестов.
> 
> > 
> > Дайте пожалуйста ответы на мои вопросы.
Comment 9 AEN 2011-04-23 11:43:01 MSK
(В ответ на комментарий №8)
> (В ответ на комментарий №6)
> > (В ответ на комментарий №4)
> > > Традицию удалять не проходящие тесты ввел "кто-то" до меня.
> > > 
> > 
> > Есть ссылка на коммит?
> 
> http://git.altlinux.org/people/raorn/packages/?p=ruby.git;a=blob;f=ruby.spec;h=7a3590cf621b730609934d1242d5c2d04bc6d489;hb=fc6177d8eac5f22d29a29c24e4893ac3714cbb8d
> 
> 
> Это SPEC из руби, который был до меня.
> Там тоже удаляются некоторые тесты.
> Строка 301.

<...>

> > > 5.
> > > Прежде чем удалят я пытался выяснить суть дела у icesik@ && raorn@....
> > > Чем история закончилась - можно догадаться.
> > > 
> > > 6.
> > > # XXX@stanv, next test doesn't passes.
> > >  rm -rf test/test_pty.rb
> > >  rm -rf test/drb/ut_array_drbssl.rb
> > >  rm -rf test/drb/test_drbssl.rb
> > >  rm -rf test/net/http/utils.rb
> > >  rm -rf test/net/http/test_https.rb
> > > 
> > > Один их тестов относится к терминалам в hasher.
> > > 
> > > Последние два теста виснут до бесконечности - поскольку я в хешере росшарил
> > > сеть.
> > Исключение тестов -- ошибка. Тем более без сообщения и обоснования в changelog.
> > Я понял Вас, спасибо. Но хочу разобраться с тем, кто все же начал удаление
> > тестов.

С этим разобрались.

Нехорошая история и я напрасно полез не разобравшись. Все равно надо к этому буде вернуться, но позже.