Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Тест в девето предизвикателство #24

Closed
AlexVPopov opened this issue Dec 19, 2013 · 10 comments
Closed

Тест в девето предизвикателство #24

AlexVPopov opened this issue Dec 19, 2013 · 10 comments

Comments

@AlexVPopov
Copy link
Contributor

Teстът:

it "memoizes results" do
  call_count = 0

  object = Object.new
  object.define_singleton_method :foo do
    call_count += 1
    :bar
  end

  memoized = Memoizer.new object

  memoized.foo.should eq :bar
  memoized.foo.should eq :bar
  call_count.should eq 1
end

в ruby 2.1.0 в момента не минава, ако се ползва public_send, понеже define_singelton_method създава private method за дадения обект. За допълнителна информация тук, тук и тук.

@mitio
Copy link
Member

mitio commented Dec 19, 2013

Благодаря за информацията!

Това изглежда като че ли е бъг в Ruby 2.1.0, който, надяваме се, ще оправят преди 2.1 final.

Тестовете, които сме пуснали на девето предизвикателство, не са гърмели заради този тест, така че там би трябвало да сме окей. Ако разбирам правилно, проблемът е, че с Ruby 2.1.0-preview2 не върви тестът в момента? Ако е само това, нямаме много какво да направим, освен да изчакаме да го оправят във final.

EDIT: Ако не съм разбрал правилно, кажи, да го отворя пак това issue.

@mitio mitio closed this as completed Dec 19, 2013
@AlexVPopov
Copy link
Contributor Author

Да, да, не е нужно да правите нищо, просто казвам. В интерес на истината, това май се е появило в preview2, защото в preview1 и аз го нямах този проблем. Просто чакаме.

@bostko
Copy link
Contributor

bostko commented Dec 19, 2013

Това е opensource. Няма нужда да чакаме, може да го фикснем :)

@gsamokovarov
Copy link

В такъв дух ви искаме!

On Thursday, December 19, 2013, Âàëüî wrote:

Òîâà å opensource. Íÿìà íóæäà äà ÷àêàìå, ìîæå äà ãî ôèêñíåì :)


Reply to this email directly or view it on GitHubhttps://github.com//issues/24#issuecomment-30943478
.

@ignisf
Copy link
Member

ignisf commented Dec 19, 2013

@gsamokovarov оправи си енкодинга

@gsamokovarov
Copy link

@ignisf Âò àêúâ äóõ âè èñê àìå?

🎅

@mitio
Copy link
Member

mitio commented Dec 20, 2013

"В такъв дух ви искаме!"

EDIT: Fixer.

@skanev
Copy link
Member

skanev commented Dec 20, 2013

@mitio: Винаги готов да напише скрипт за проблем, решен отдавна:

http://2cyr.com/decode/

@gsamokovarov
Copy link

Човек да не тролне 😄

@mitio
Copy link
Member

mitio commented Dec 21, 2013

@skanev, много готин сайт, благодаря :) Сигурен бях, че има нещо готово, но друго си е човек да се почеше, да помисли какво е станало и да намери начин да го оправи със собствения си чук 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants