Repository navigation
2.7.1: eski erişimciler eski uca sabitlendi - #4
Merged
Merged
Conversation
client.ai ve client.mockups 2.7.0'da aile yollarına yönlendirilmişti. Paketi yükseltip kodunu değiştirmeyen çağıranın açtığı yeni işler sessizce photo_mockup_* kind'ıyla dönüyor, job.kind'a göre dallanan kod hiçbir hata vermeden eşleşmez oluyordu. İkisi de yeniden /api/v1/sudoai/2d-mockups ve /api/v1/mockups üzerinde; photoMockups ve psdMockups aile yollarında kalıyor. İki eski erişimci setter kazandı. Getter-only olduklarından test sahtesi enjekte eden atama ES modülünde TypeError atıyor, düz CommonJS'te (sloppy mod) sessizce düşüyordu; ikincisinde çağrı testten çıkıp gerçek API'ye gidiyor ve gerçek kredi harcıyordu. Testler iki dalı da kanıtlıyor: eski erişimcinin her yöntemi kendi yolunda, atama ise hem strict hem sloppy modda.
…tiliyor client.ai / client.mockups kullanımdan kaldırma uyarısı iki adı "aynı nesne" diye anlatıyordu. 2.7.1'den beri doğru değil: her ad kendi yayımlandığı yolda duruyor, eski ad 2d_* kind'larını, aile adı photo_mockup_* kind'larını döndürüyor. "Bedava takas" okuması, uyarıyı okuyup adı değiştiren çağıranı tam da 2.7.1'in kapattığı sessiz kind değişikliğine sokuyordu; uyarı artık yolu söylüyor. WebhookEventNaming ve CreateWebhookEndpointParams.eventNaming "yeni uç her zaman current'a sabitlenir" diyordu. Pin gönderilmediğinde varsayılan artık event_types yazımından türetiliyor: aile adları current, eski adlar legacy, boş ya da karışık liste legacy. Tip açıklamaları, README ve CHANGELOG bunu anlatıyor. Bu pakette davranış değişmedi; anlatılan API'nin davranışı. Eski erişimcilerde getter/setter simetrisi bekçi testle kayda geçti: setter hangi alanı yazıyorsa getter onu okuyor, aile erişimcisine dokunulmuyor, bu yüzden kaydet-geri yükle turu iki erişimciyi de başlangıç haline döndürüyor. Setter'ı aile alanını da yazacak şekilde bozan mutasyonda iki tur testi de kırılıyor. vi.spyOn geri alması ve instance üstünden delete aynı testte.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Yükseltip kodunu değiştirmeyen kullanıcı için davranış sabit kalır: eski erişimciler eski adresi çağırır. Eski erişimciye atama düz JavaScript'te de sessizce düşmez. Kullanımdan kaldırma uyarısının metni gerçeği anlatır.