Skip to content

Test/search#214

Closed
Alyaa242 wants to merge 86 commits intomainfrom
test/search
Closed

Test/search#214
Alyaa242 wants to merge 86 commits intomainfrom
test/search

Conversation

@Alyaa242
Copy link
Copy Markdown
Collaborator

@Alyaa242 Alyaa242 commented Dec 15, 2025

No description provided.

Alyaa242 and others added 30 commits December 8, 2025 13:03
@Alyaa242
fix(search): remove irrelevant posts
739b398
@MoBahgat010
Fix/notifications (#150)
c2fed33 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response
@MoBahgat010
Fix/notifications (#155)
c8453db 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug
@Alyaa242
feat(search): add trends to search
a8eca65
@Alyaa242
fix(search): add is_bookmarked to search response
b5c00f3
@Alyaa242
fix(search): fix quotes parent tweets
30822b1
@Alyaa242
fix(search): fix updating quotes and replies bg job
fa4f938
@Alyaa242
feat(search): mention suggestions
08efd86
@MoBahgat010
Fix/notifications (#156)
121a748 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): ya rab el aggregation y4t8l b2a
@Alyaa242
refactor(search): remove unnecessary user index logic
10b7900
@MoBahgat010
Fix/notifications (#157)
f6f4036 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): ya rab el aggregation y4t8l b2a

* fix(notifications): el data kter wel 5er kteer
@MoBahgat010
Fix/notifications (#158)
4022ad5 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): ya rab el aggregation y4t8l b2a

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer
@MoBahgat010
Fix/notifications (#159)
70ae42a 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): ya rab el aggregation y4t8l b2a

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer
@MoBahgat010
Fix/notifications (#160)
1b7b2bc 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): ya rab el aggregation y4t8l b2a

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer
@Alyaa242
fix(search): remove trigram user indexes
ea754be
@AmiraKhalid04
fix(trends): use built in function for hashtag extraction
c672a2a
@AmiraKhalid04
fix(hashtags): convert hashtag names to lowercase
568a7aa
@Alyaa242
refactor(search): organize posts search functionalities
5486b7c
@Alyaa242
refactor(search): organize users search functionalities
ed823e5
@AmiraKhalid04
fix(trends): adjust candidates TTL
5120430
@MoBahgat010
Fix/notifications (#163)
6246061 
* fix(notifications): fix some bugs

* fix(notifications): fix some bugs

* fix(notifications): sort response

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): fix aggregation bug

* fix(notifications): ya rab el aggregation y4t8l b2a

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): el data kter wel 5er kteer

* fix(notifications): expo

* fix(notifications): expo

* fix(notifications): expo
@AmiraKhalid04
refactor(trends): remove debugging params
5994a10
@AmiraKhalid04
fix(trends): edit hashtag regex
6ded9c6
@AmiraKhalid04
refactor(trends): move cron expression to queue constants file
2576835
@shady-2004
Merge branch 'dev' into feat/trends-v3
2a13469
@AmiraKhalid04
fix(dependencies): resolve package lock error
acb02b6
@AmiraKhalid04
refactor(trends): use built in cron expression constants
7837c2b
@AmiraKhalid04
test(trends): fix unit tests
c6097c1
@AmiraKhalid04 @shady-2004
fix(trends): use built in function for hashtag extraction (#162)
5393689 
* fix(trends): use built in function for hashtag extraction

* fix(hashtags): convert hashtag names to lowercase

* fix(trends): adjust candidates TTL

* refactor(trends): remove debugging params

* fix(trends): edit hashtag regex

* refactor(trends): move cron expression to queue constants file

* fix(dependencies): resolve package lock error

* refactor(trends): use built in cron expression constants

* test(trends): fix unit tests

---------

Co-authored-by: shady <149704119+shady-2004@users.noreply.github.com>
@AmiraKhalid04
fix(trends): make category case insensitive
cd8a70c
MoBahgat010 and others added 25 commits December 13, 2025 13:19
@MoBahgat010
Fix/notification response (#188)
8698656 
* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): extra data

* fix(notifications): extra data

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): not important

* fix(notifications): send quote notification only if tweet owner is already mentioned in it
@shady-2004
fix(bookmarks): is_bookmarked
3f83172
@Alyaa242
fix(search): fix minimum should match
a9fbe72 
* fix(search): fix minimum should match

* fix(search): fix suggestions

* test(search): fix suggestions unit test
@AmiraKhalid04
fix(trends): fix category condition (#190)
c33d9ee
@Alyaa242
fix(profile): add reposted_by (#191)
528a53a 
* fix(profile): add reposted_by

* fix(profile): remove migrations
@AmiraKhalid04
fix(clean-job): remove duplicated decrement code (#192)
550b4c1
@MoBahgat010
Fix/notification response (#193)
30d894e 
* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): reply original tweet data

* fix(notifications): extra data

* fix(notifications): extra data

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): push notification format

* fix(notifications): not important

* fix(notifications): send quote notification only if tweet owner is already mentioned in it

* fix(notifications): fix mention in push notification

* fix(notifications): fix mention in push notification

* fix(notifications): fix mention in push notification

* fix(notifications): fix mention in push notification
@Alyaa242
Fix/profile v4 (#194)
716c75d 
* fix(profile): add reposted_by

* fix(profile): remove migrations

* fix(profile): add username to reposted by
@AmiraKhalid04 @MarioRaafat
Fix/hashtag count (#187)
035ea10 
* fix(hashtags): remove created by column

* feat(hashtag): add tweet hashtag entity

* feat(trend): disable fake trends for now

* feat(hashtags): add trigger for hashtag count

* feat(hashtags): extract hashtags from quotes and replies

* fix(db): remove view migrations

* fix(db): copy  migrations file to database folder

* test(hashtag): fix unit tests

---------

Co-authored-by: Mario Raafat <136023677+MarioRaafat@users.noreply.github.com>
@MoBahgat010
fix(notifications): ya rab el 5alaaaaaas (#196)
ba622e1
@Alyaa242
fix(profile): fix parent tweet for quotes and replies (#195)
44a8ecf 
* fix(profile): fix parent tweet for quotes and replies

* test(tweets): fix tweets repository unit tests
@MoBahgat010
fix(notifications): er7mny b2a ya saleh (#197)
a0bf455 
* fix(notifications): er7mny b2a ya saleh

* fix(notifications): er7mny b2a ya saleh
@AmiraKhalid04
fix(trends): remove hashtags with count zero (#198)
c1e5e0a
@AmiraKhalid04
Feat/fake trends (#200)
9b464c4 
* feat(trend): add trend seed

* feat(trend): update trend seed

* feat(trend): insert in batches into redis

* feat(db-migrations): update migration folder

* test(trend): fix unit tests
@shady-2004
fix(explore): clear endpoint (#202)
cafbd90
@AmiraKhalid04
fix(timeline): fix pagination by id in for you
f590ff6
@Alyaa242
Fix/search v4 (#201)
ad6394d 
* fix(search): filter out tweets with deleted parents

* test(search): add more unit tests

* refactor(search): elasticsearch delete tweets job in a batch
@AmiraKhalid04
Feat/fake trends v2 (#203)
cc2dcf6 
* feat(trend): increase TTL time for trend just for testing scenarios

* test(trend): fix unit tests
@AmiraKhalid04
Feat/fake trends v2 (#204)
6a86abe 
* feat(trend): increase TTL time for trend just for testing scenarios

* test(trend): fix unit tests

* feat(trend): add  trigger endpoint for calculating trends
@AmiraKhalid04
fix(timeline): add conversation user id and parent user id to view (#205
f2ce18a 
)
@shady-2004
test(sonar): setup sonar (#206)
d9518aa
@Alyaa242
fix(auth): fix access tokens on delete account (#207)
94ef2a8 
* fix(auth): fix access tokens on delete account

* test(user): fix user tests
@Alyaa242
fix(tweets): fix es delete tweets bg job (#208)
6b98d62
@Alyaa242
test(search): test elasticsearch module
24f588b
@Alyaa242
test(search): add search test cases
410b6f3
@Alyaa242 Alyaa242 closed this Dec 15, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Dec 15, 2025

Coverage after merging test/search into main will be

76.81%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   app.controller.ts87.80%66.67%100%100%19, 33, 88, 88, 88
   app.module.ts100%100%100%100%
   app.service.ts90%71.43%100%100%101, 122–123, 123, 195, 21, 21, 23, 242, 25, 250, 27, 29–30, 330, 339, 349, 363, 372, 380
src/auth
   auth.controller.ts80.11%68.90%87.50%89.20%102, 105, 107, 110, 110, 122, 133, 145, 160, 175, 201, 213, 224, 237, 253, 273, 291, 310, 314–316, 329, 352, 367, 382, 396, 420, 462–464, 469–470, 470, 475–478, 478, 490, 495, 520–522, 527–528, 528, 533–536, 536, 547, 552, 577, 618–620, 625–626, 626, 631–634, 634, 645, 650, 663, 675, 697
   auth.module.ts92.31%100%0%96%30
   auth.service.ts89.56%81.65%94.29%93.93%107–108, 1085, 143, 432, 480, 510, 543–544, 56, 56, 565–566, 57–58, 589, 59, 590, 60–62, 646, 651, 673, 70, 704, 708, 71, 728, 756, 77, 775, 78, 797–798, 802–803, 811, 811, 811–812, 815, 822, 822, 822–823, 828, 828, 828–829, 832, 837–838, 838, 838–839, 842, 844, 844, 844–845, 848, 848, 848–849, 878–879, 885, 888–889, 981, 981, 990, 990, 992–993
   captcha.service.ts96.30%90%100%100%16, 69
   username.service.ts94.50%86.84%100%98.44%112, 115, 42, 47, 7, 77
src/auth/guards
   facebook.guard.ts100%100%100%100%
   github.guard.ts100%100%100%100%
   google-auth.guard.ts100%100%100%100%
   jwt.guard.ts48.78%56.25%66.67%40.91%12, 14, 14, 14–15, 18–19, 21–22, 22, 22–24, 26, 28, 28, 28–29, 33, 7
   optional-jwt.guard.ts84.62%100%50%85.71%8
   ws-jwt.guard.ts96.61%90%100%100%16, 52
src/auth/strategies
   facebook.strategy.ts88.41%79.49%100%100%11, 11, 15–17, 36–37, 37
   github.strategy.ts89.23%81.08%100%100%11, 11, 15–17, 39, 39
   google.strategy.ts92%83.33%100%100%10, 10, 14–15
   jwt.strategy.ts92.31%75%100%100%8
src/azure-storage
   azure-storage.module.ts100%100%100%100%
   azure-storage.service.ts98.04%92.86%100%100%11
src/background-jobs
   background-jobs.module.ts96.43%100%0%98.18%63
   background-jobs.ts95.74%0%100%100%16–17
src/background-jobs/ai-summary
   ai-summary.processor.ts81.63%71.43%100%84.85%19, 21, 63, 63, 63, 66, 71, 73–74
   ai-summary.service.ts100%100%100%100%
src/background-jobs/elasticsearch
   es-delete-tweet.service.ts100%100%100%100%
   es-delete-user.service.ts100%100%100%100%
   es-follow.service.ts100%100%100%100%
   es-index-tweet.service.ts100%100%100%100%
   es-sync.processor.ts90.61%82.72%93.75%95.69%100, 103, 149–150, 23, 23, 242, 25, 258, 26, 32, 348, 357, 362, 364, 84–86, 99
   es-update-user.service.ts100%100%100%100%
src/background-jobs/email
   email.controller.ts96.15%75%100%100%9
   email.processor.ts97.67%91.67%100%100%18
   email.service.ts100%100%100%100%
src/background-jobs/explore
   explore-jobs.controller.ts86.36%75%75%92.86%10, 45
   explore-jobs.cron.ts91.30%83.33%100%93.33%15, 42
   explore-jobs.processor.ts95.29%80%100%100%101, 12, 155, 18
   explore-jobs.service.ts45.58%34.04%55.56%48%119, 179, 188–190, 209–211, 213, 215–216, 216, 216, 218, 220–221, 223–224, 229, 229, 229, 231–232, 236, 238–239, 249, 252, 254, 254, 254–256, 260–263, 271, 274, 274, 274–276, 280–281, 283–285, 287, 287, 287, 287, 287, 287, 287–289, 292–297, 300, 303, 307, 307, 307–309, 313–314, 317–318, 320–321, 321, 321–322, 324–325, 327, 327, 327, 329, 33, 33, 330, 334, 337–338, 338, 338–339, 342, 342, 342–344, 347, 353, 356, 358–359, 364, 425–426, 89–90
src/background-jobs/hashtag
   hashtag.controller.ts96.15%75%100%100%8
   hashtag.processor.ts90.91%75%100%100%12, 15
   hashtag.service.ts96.15%87.50%100%100%10
src/background-jobs/notifications/clear
   clear.processor.ts85.71%82.35%100%86.96%12, 26, 34–37
   clear.service.ts70%50%66.67%88.89%38, 40, 40–41, 41
src/background-jobs/notifications/follow
   follow.processor.ts92.11%75%100%100%18, 18, 35
   follow.service.ts100%100%100%100%
src/background-jobs/notifications/like
   like.processor.ts94.44%85%100%100%19, 19–20
   like.service.ts100%100%100%100%
src/background-jobs/notifications/mention
   mention.processor.ts94.85%88.37%100%100%23, 23–25, 77
   mention.service.ts100%100%100%100%
src/background-jobs/notifications/message
   message.processor.ts92.86%78.57%100%100%19, 19–20
   message.service.ts100%100%100%100%
src/background-jobs/notifications/quote
   quote.processor.ts94.74%86.67%100%100%23, 23–25
   quote.service.ts100%100%100%100%
src/background-jobs/notifications/reply
   reply.processor.ts90.54%83.33%100%95.24%20, 20–22, 82–84
   reply.service.ts100%100%100%100%
src/background-jobs/notifications/repost
   repost.processor.ts94.29%85.71%100%100%

@MarioRaafat MarioRaafat deleted the test/search branch December 24, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MarioRaafat MarioRaafat Awaiting requested review from MarioRaafat

@shady-2004 shady-2004 Awaiting requested review from shady-2004

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Alyaa242 @MoBahgat010 @AmiraKhalid04 @shady-2004 @MarioRaafat