Skip to content

feat: temporarily disable Twitter/X source#8

Open
4ndreello wants to merge 1 commit into
mainfrom
disable-twitter-temp
Open

feat: temporarily disable Twitter/X source#8
4ndreello wants to merge 1 commit into
mainfrom
disable-twitter-temp

Conversation

@4ndreello

Copy link
Copy Markdown
Owner

Desabilita temporariamente o Twitter/X do feed por instabilidade da API/fonte.

Mudanças

  • Remove chamadas do Twitter do pipeline do SmartMix (fetch, enrich, rank, interleave, persist)
  • Marca Twitter como pausado no status de fontes do feed
  • Corrige mock do TwitterApi nos testes (arrow function não era construtível)

Como reativar

Basta restaurar as linhas removidas em e o item em de . O serviço e seus testes permanecem intactos.

Verificações

  • [x]
    �[1m�[46m RUN �[49m�[22m �[36mv4.0.15 �[39m�[90m/home/andreello/Desktop/dev/tech-news-api�[39m

�[90mstdout�[2m | src/services/ranking.service.test.ts�[2m > �[22m�[2mRankingService�[2m > �[22m�[2mshould penalize posts with 0 comments
�[22m�[39m{ scoreWithComments: �[33m152�[39m, scoreNoComments: �[33m25�[39m }

�[90mstdout�[2m | src/services/ranking.service.test.ts�[2m > �[22m�[2mRankingService�[2m > �[22m�[2mshould favor older posts with engagement over new posts with zero engagement
�[22m�[39m{ oldScore: �[33m32�[39m, newScore: �[33m2�[39m }

�[32m✓�[39m src/services/ranking.service.test.ts �[2m(�[22m�[2m3 tests�[22m�[2m)�[22m�[32m 6�[2mms�[22m�[39m
�[32m✓�[39m src/services/twitter.service.test.ts �[2m(�[22m�[2m3 tests�[22m�[2m)�[22m�[32m 5�[2mms�[22m�[39m

�[2m Test Files �[22m �[1m�[32m2 passed�[39m�[22m�[90m (2)�[39m
�[2m Tests �[22m �[1m�[32m6 passed�[39m�[22m�[90m (6)�[39m
�[2m Start at �[22m 17:27:17
�[2m Duration �[22m 329ms�[2m (transform 152ms, setup 0ms, import 316ms, tests 11ms, environment 0ms)�[22m passa

  • passa

- Remove Twitter from SmartMix fetch/enrich/rank/interleave/persist pipeline
- Mark Twitter as disabled in FeedService source status
- Fix TwitterService mock in test (arrow function broke constructor)
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.

1 participant