Skip to content

Fix/release 3.9.1#4932

Closed
vrrayz wants to merge 56 commits intoJoystream:mainfrom
vrrayz:fix/release-3.9.1
Closed

Fix/release 3.9.1#4932
vrrayz wants to merge 56 commits intoJoystream:mainfrom
vrrayz:fix/release-3.9.1

Conversation

@vrrayz
Copy link
Contributor

@vrrayz vrrayz commented Dec 3, 2025

No description provided.

thesan and others added 30 commits August 10, 2024 21:43
* Replace Subscan with Statescan

* Fix formatting

* Fix failing playwright install in CI checks
…m#4869)

* react snap setup

* react snap setup

* react snap setup

* seo

* council and past council page

* Add term start and end (estimated) dates for current term

* total number of votes & total voting stake during Voting and Revealing period

* Add Election Result and Revealed votes statistics

* added past councils route and candidate vote highlight

* latest election fields update

* undo react-snap

* removed uneccesary package update

* removed uneccesary package update

* chore: remove yarn.lock from tracked files

* git error fix

* chore: revert yarn.lock to match main

* removed eslint comment

Co-authored-by: Leszek Wiesner <[email protected]>

* Apply suggestions from code review

Co-authored-by: Leszek Wiesner <[email protected]>

* Update packages/ui/src/index.html

Co-authored-by: Leszek Wiesner <[email protected]>

* Update packages/ui/src/council/components/pastCouncil/PastCouncilsList/PastCouncilListItem.tsx

Co-authored-by: Leszek Wiesner <[email protected]>

* Update packages/ui/src/council/components/election/pastElection/PastElectionStats.tsx

Co-authored-by: Leszek Wiesner <[email protected]>

* estimated ends at

* council id

* rearranged mock

* review changes

* review changes

* test error fix

---------

Co-authored-by: Leszek Wiesner <[email protected]>
* Replace Subscan with Statescan (Joystream#4867)

* Replace Subscan with Statescan

* Fix formatting

* Fix failing playwright install in CI checks

* Bump version to 3.7.3

---------

Co-authored-by: Theophile Sandoz <[email protected]>
* Replace Subscan with Statescan (Joystream#4867)

* Replace Subscan with Statescan

* Fix formatting

* Fix failing playwright install in CI checks

* Council and election pages: Small enhancements - Issue 4866 (Joystream#4869)

* react snap setup

* react snap setup

* react snap setup

* seo

* council and past council page

* Add term start and end (estimated) dates for current term

* total number of votes & total voting stake during Voting and Revealing period

* Add Election Result and Revealed votes statistics

* added past councils route and candidate vote highlight

* latest election fields update

* undo react-snap

* removed uneccesary package update

* removed uneccesary package update

* chore: remove yarn.lock from tracked files

* git error fix

* chore: revert yarn.lock to match main

* removed eslint comment

Co-authored-by: Leszek Wiesner <[email protected]>

* Apply suggestions from code review

Co-authored-by: Leszek Wiesner <[email protected]>

* Update packages/ui/src/index.html

Co-authored-by: Leszek Wiesner <[email protected]>

* Update packages/ui/src/council/components/pastCouncil/PastCouncilsList/PastCouncilListItem.tsx

Co-authored-by: Leszek Wiesner <[email protected]>

* Update packages/ui/src/council/components/election/pastElection/PastElectionStats.tsx

Co-authored-by: Leszek Wiesner <[email protected]>

* estimated ends at

* council id

* rearranged mock

* review changes

* review changes

* test error fix

---------

Co-authored-by: Leszek Wiesner <[email protected]>

* Bump version, update CHANGELOG

---------

Co-authored-by: Theophile Sandoz <[email protected]>
Co-authored-by: Victor Emmanuel <[email protected]>
Co-authored-by: Oleksandr Korniienko <[email protected]>
Co-authored-by: Theophile Sandoz <[email protected]>
@vrrayz vrrayz closed this Dec 3, 2025
@vrrayz vrrayz reopened this Dec 3, 2025
@vrrayz vrrayz changed the base branch from dev to main December 3, 2025 14:09
@traumschule
Copy link
Collaborator

Thanks! Merged, two are still flaky

FAIL test/proposals/hooks/useProposals.test.tsx (933 MB heap size)
  ● useProposals › Status: active | past › Status: active

    expect(received).toBeTruthy()

    Received: false

      33 |       expect(result.proposals.length).toBeGreaterThan(0)
      34 |       result.proposals.forEach((proposal) => {
    > 35 |         expect(proposalActiveStatuses.includes(proposal.status)).toBeTruthy()
         |                                                                  ^
      36 |       })
      37 |     })
      38 |

      at forEach (test/proposals/hooks/useProposals.test.tsx:35:66)
          at Array.forEach (<anonymous>)
      at Object.<anonymous> (test/proposals/hooks/useProposals.test.tsx:34:24)

FAIL test/forum/modals/CreatePostModal.test.tsx (6.892 s, 895 MB heap size)
  ● UI: CreatePostModal › Transaction success

    Unable to find an element with the text: Your post has been submitted.. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

    Ignored nodes: comments, <script />, <style />
    <body>
      <div />
    </body>

      117 |     })
      118 |
    > 119 |     expect(await screen.findByText('Your post has been submitted.')).toBeDefined()
          |                         ^
      120 |   })
      121 |
      122 |   it('Displays post deposit', () => {

      at waitForWrapper (../../node_modules/@testing-library/react/node_modules/@testing-library/dom/dist/wait-for.js:187:27)
      at findByText (../../node_modules/@testing-library/react/node_modules/@testing-library/dom/dist/query-helpers.js:101:32)
      at Object.<anonymous> (test/forum/modals/CreatePostModal.test.tsx:119:25)


Test Suites: 2 failed, 1 skipped, 111 passed, 113 of 114 total
Tests:       2 failed, 4 skipped, 701 passed, 707 total

@vrrayz
Copy link
Contributor Author

vrrayz commented Dec 3, 2025

Thanks! Merged, two are still flaky

FAIL test/proposals/hooks/useProposals.test.tsx (933 MB heap size)
  ● useProposals › Status: active | past › Status: active

    expect(received).toBeTruthy()

    Received: false

      33 |       expect(result.proposals.length).toBeGreaterThan(0)
      34 |       result.proposals.forEach((proposal) => {
    > 35 |         expect(proposalActiveStatuses.includes(proposal.status)).toBeTruthy()
         |                                                                  ^
      36 |       })
      37 |     })
      38 |

      at forEach (test/proposals/hooks/useProposals.test.tsx:35:66)
          at Array.forEach (<anonymous>)
      at Object.<anonymous> (test/proposals/hooks/useProposals.test.tsx:34:24)

FAIL test/forum/modals/CreatePostModal.test.tsx (6.892 s, 895 MB heap size)
  ● UI: CreatePostModal › Transaction success

    Unable to find an element with the text: Your post has been submitted.. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.

    Ignored nodes: comments, <script />, <style />
    <body>
      <div />
    </body>

      117 |     })
      118 |
    > 119 |     expect(await screen.findByText('Your post has been submitted.')).toBeDefined()
          |                         ^
      120 |   })
      121 |
      122 |   it('Displays post deposit', () => {

      at waitForWrapper (../../node_modules/@testing-library/react/node_modules/@testing-library/dom/dist/wait-for.js:187:27)
      at findByText (../../node_modules/@testing-library/react/node_modules/@testing-library/dom/dist/query-helpers.js:101:32)
      at Object.<anonymous> (test/forum/modals/CreatePostModal.test.tsx:119:25)


Test Suites: 2 failed, 1 skipped, 111 passed, 113 of 114 total
Tests:       2 failed, 4 skipped, 701 passed, 707 total

Its been a while so i can't remember if active and past are part of the values for the status key and from the issue i checked the mock data testProposals and noticed it doesn't have proposals with status:active hence why it would always return false.

in packages\ui\src\proposals\hooks\useProposals.ts there's this type UseProposalsStatus = 'active' | 'past'
but in the mock data for testProposals there are different statuses and none of them make use of UseProposalsStatus type

The second case happens because src\forum\modals\PostActionModal\CreatePostModal didn't have the success message set here const [state, send] = useMachine( defaultTransactionModalMachine('There was a problem posting your message.', undefined), { context: { validateBeforeTransaction: true } } )

@traumschule
Copy link
Collaborator

Thanks again. Since the test started failing since #4887 I assume we should remove vetoed proposals from the mock data in test/_mocks/proposals/index.ts.. Closing this and will apply fix in #4927.

@traumschule traumschule closed this Dec 3, 2025
traumschule added a commit to traumschule/pioneer that referenced this pull request Dec 3, 2025
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

Successfully merging this pull request may close these issues.

6 participants