Skip to content

refactor: Federation refactor towards services + file names improvement + E2E tests fixes#28326

Merged
kodiakhq[bot] merged 12 commits into
developfrom
feat/federation-6.1
Mar 10, 2023
Merged

refactor: Federation refactor towards services + file names improvement + E2E tests fixes#28326
kodiakhq[bot] merged 12 commits into
developfrom
feat/federation-6.1

Conversation

@MarcosSpessatto

@MarcosSpessatto MarcosSpessatto commented Mar 8, 2023

Copy link
Copy Markdown
Contributor

Proposed changes (including videos or screenshots)

This PR is the one that groups the other 4 PRs related to refactoring federation moving from the regular "app" folder to the new services implementation, it also refactors some files and folders to more meaningful and finally it fix some E2E tests.

#28263
#28267
#28295
#28158

Issue(s)

Steps to test or reproduce

Further comments

Marcos Spessatto Defendi added 4 commits February 28, 2023 13:38
…ase (#28158)

* fix: fix e2e tests based on selector changes made in the latest release

* fix: fix more tests
* chore: move federation to an internal service

* fix: fix tests

* test: fix test
* chore: move federation to an internal service

* fix: fix tests

* chore: move federation from app folder to server

* test: fix test

* fix: fix mistakes from merge
@codecov

codecov Bot commented Mar 8, 2023

Copy link
Copy Markdown

Codecov Report

Merging #28326 (aad0a0a) into develop (78ccb57) will increase coverage by 12.70%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           develop   #28326       +/-   ##
============================================
+ Coverage    32.62%   45.33%   +12.70%     
============================================
  Files          621      759      +138     
  Lines        12368    14782     +2414     
  Branches      1863     2083      +220     
============================================
+ Hits          4035     6701     +2666     
+ Misses        8135     7789      -346     
- Partials       198      292       +94     
Flag Coverage Δ
e2e 45.29% <ø> (+12.66%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

* chore: move federation to an internal service

* fix: fix tests

* chore: move federation from app folder to server

* chore: improve folder names and file names

* test: fix test

* refactor: rename class property

* fix: fix tests
@MarcosSpessatto MarcosSpessatto changed the title Chore: Federation refactor Chore: Federation refactor towards services + file names improvement + E2E tests fixes Mar 8, 2023
@MarcosSpessatto MarcosSpessatto self-assigned this Mar 8, 2023
@MarcosSpessatto MarcosSpessatto requested a review from lmauromb March 8, 2023 11:55
@MarcosSpessatto MarcosSpessatto added this to the 6.1.0 milestone Mar 8, 2023
@MarcosSpessatto MarcosSpessatto marked this pull request as ready for review March 8, 2023 12:01
@MarcosSpessatto MarcosSpessatto requested a review from a team as a code owner March 8, 2023 12:01
lmauromb
lmauromb previously approved these changes Mar 8, 2023
@MarcosSpessatto MarcosSpessatto added the area: backend Touches the code on backend side label Mar 10, 2023
@dionisio-bot dionisio-bot Bot added the stat: ready to merge PR tested and approved waiting for merge label Mar 10, 2023
@MarcosSpessatto MarcosSpessatto removed area: backend Touches the code on backend side stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@dionisio-bot dionisio-bot Bot added the stat: ready to merge PR tested and approved waiting for merge label Mar 10, 2023
@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@MarcosSpessatto MarcosSpessatto changed the title Chore: Federation refactor towards services + file names improvement + E2E tests fixes Refactor: Federation refactor towards services + file names improvement + E2E tests fixes Mar 10, 2023
@MarcosSpessatto MarcosSpessatto changed the title Refactor: Federation refactor towards services + file names improvement + E2E tests fixes Chore: Federation refactor towards services + file names improvement + E2E tests fixes Mar 10, 2023
@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@MarcosSpessatto MarcosSpessatto changed the title Chore: Federation refactor towards services + file names improvement + E2E tests fixes refactor: Federation refactor towards services + file names improvement + E2E tests fixes Mar 10, 2023
@dionisio-bot dionisio-bot Bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Mar 10, 2023
@kodiakhq kodiakhq Bot merged commit 7b7b8a3 into develop Mar 10, 2023
@kodiakhq kodiakhq Bot deleted the feat/federation-6.1 branch March 10, 2023 19:51
gabriellsh added a commit that referenced this pull request Mar 13, 2023
…ketChat/Rocket.Chat into matrixSearchTestBranch

* 'feat/federation-public-room-search-2' of github.com:RocketChat/Rocket.Chat: (42 commits)
  refactor: Replace `_.isObject` with `isObject` (#28376)
  fix: small tweaks
  chore: Remove old css (#28140)
  fix: Thread list not scrolling to bottom when sending a message. (#28346)
  refactor: Federation refactor towards services + file names improvement + E2E tests fixes (#28326)
  i18n: Language update from LingoHub 🤖 on 2023-03-10Z (#28365)
  refactor: Improve sidebar search list hook (#28328)
  Chore: Change the PR title check to use conventional commits (#28358)
  [IMPROVE] Galician onboarding translation (#27908)
  [IMPROVE] Adding input type=password to AWS secrets fields (#28159)
  [FIX] Normal and multiline messages adding extra spaces upon copying/pasting (#28187)
  Chore: remove unused prop in register workspace modal (#28317)
  [FIX] App crashing when switching to Hindi (hi-In) language (#28285)
  [NEW] Registered Loom oEmbed (#28339)
  fix: create dm users before join the room
  tests: add unit tests + small tweaks
  fix: fix invite links button logic
  chore: fix lint
  Chore: Remove/replace some Meteor packages (#28189)
  Chore: Unused/dead code removal (#28155)
  ...
@casalsgh casalsgh modified the milestones: 6.1.0, 6.2.0 Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants