Page MenuHomeSealhub

Move collection fields to root in response
ClosedPublic

Authored by piotr-ptaszynski on Aug 26 2018, 12:02.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 04:16
Unknown Object (File)
Wed, Mar 27, 04:16
Unknown Object (File)
Wed, Mar 27, 04:16
Unknown Object (File)
Wed, Mar 27, 02:07
Unknown Object (File)
Mon, Mar 25, 09:16
Unknown Object (File)
Fri, Mar 22, 02:58
Unknown Object (File)
Thu, Mar 14, 10:49
Unknown Object (File)
Tue, Mar 5, 10:58
Subscribers
None

Diff Detail

Repository
rS Sealious
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Checklista diffów do sealiousa:

  • EJKO działa poprawnie (make test + testy manualne) - jeżeli nie, zrobić odpowiedniego diffa do EJKO
  • sealious-template działa poprawnie (make test)- jeżeli nie, zrobić odpowiedniego diffa do sealious-template
  • aktualizacja dokumentacji
  • czy wszystko nadal się zgadza w getting_started
kuba-orlik added a child revision: Restricted Differential Revision.Aug 29 2018, 15:22

Kilka uwag (poza komentarzami inline):

  1. Testy zwracają błąd:
114 passing (33s)
1 failing

1) special filters
     Matches
       returns only positive numbers when using filter:

    AssertionError [ERR_ASSERTION]: [ 2, 1 ] deepEqual [ 1, 2 ]
    + expected - actual

     [
    +  1
       2
    -  1
     ]
    
    at with_running_app (lib/app/base-chips/special_filters/matches.subtest.js:43:11)
  1. Trzeba zaktualizować getting_started - odpowiedź HTTP w ostatnim przykładzie (na samym dole) zawiera body:

https://hub.sealcode.org/source/sealious/browse/alpha/getting_started/getting_started.md;4d69badf27290080929ac379586b3d0334456df8?as=remarkup

lib/app/base-chips/access-strategy-types/same-as-for-resource-in-field.subtest.js
123 ↗(On Diff #1244)

To chyba jest niepożądane ;)

This revision now requires changes to proceed.Aug 29 2018, 15:43

I jeszcze jedna rzecz - uruchomienie testów na sealious-template z patchem {D346} i podlinkowanym sealiousem z D350: Move collection fields to root in response daje error:

0 passing (2s)
1 failing

1) Collection users returns test user:
   TypeError: Cannot read property 'username' of undefined
    at Context.it (src/backend/collections/users.test.js:9:48)
    at process._tickCallback (internal/process/next_tick.js:68:7)

Wiem z czego się biorą te różne rzeczy, ale poczekam aż wylądujesz D346

Wiem z czego się biorą te różne rzeczy, ale poczekam aż wylądujesz D346

Done ;)

  • Remove race condition vulnerabilities in tests
  • Update getting started
  • Merge branch 'alpha' into piopta_move_collection_field_to_root
kuba-orlik added a child revision: Restricted Differential Revision.Sep 12 2018, 18:38
This revision is now accepted and ready to land.Sep 18 2018, 16:26
This revision was automatically updated to reflect the committed changes.
kuba-orlik changed the visibility from "All Users" to "Public (No Login Required)".Feb 21 2019, 11:13