Skip to content

๐Ÿ”จ [Refactoring] Admin ๊ณต์œ ์ผ์ • ์กฐํšŒ API -> ๋ถ„๋ฅ˜๋ณ„ ์ผ์ •์กฐํšŒ API๋กœ ๋ณ€๊ฒฝ #1091#1093

Open
seyeon22222 wants to merge 5 commits into
devfrom
1091-refactoring-admin-๊ณต์œ ์ผ์ •-์กฐํšŒ-api---๋ถ„๋ฅ˜๋ณ„-์ผ์ •์กฐํšŒ-api๋กœ-๋ณ€๊ฒฝ

Hidden character warning

The head ref may contain hidden characters: "1091-refactoring-admin-\uacf5\uc720\uc77c\uc815-\uc870\ud68c-api---\ubd84\ub958\ubcc4-\uc77c\uc815\uc870\ud68c-api\ub85c-\ubcc0\uacbd"
Open

๐Ÿ”จ [Refactoring] Admin ๊ณต์œ ์ผ์ • ์กฐํšŒ API -> ๋ถ„๋ฅ˜๋ณ„ ์ผ์ •์กฐํšŒ API๋กœ ๋ณ€๊ฒฝ #1091#1093
seyeon22222 wants to merge 5 commits into
devfrom
1091-refactoring-admin-๊ณต์œ ์ผ์ •-์กฐํšŒ-api---๋ถ„๋ฅ˜๋ณ„-์ผ์ •์กฐํšŒ-api๋กœ-๋ณ€๊ฒฝ

Conversation

@seyeon22222

Copy link
Copy Markdown
Contributor

๐Ÿ“Œ ๊ฐœ์š”

  • Admin ๊ณต์œ ์ผ์ • ์กฐํšŒ API -> ๋ถ„๋ฅ˜๋ณ„ ์ผ์ •์กฐํšŒ API๋กœ ๋ณ€๊ฒฝํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿ’ป ์ž‘์—…์‚ฌํ•ญ

  • Admin ๊ณต์œ ์ผ์ • ์กฐํšŒ API -> ๋ถ„๋ฅ˜๋ณ„ ์ผ์ •์กฐํšŒ API๋กœ ๋ณ€๊ฒฝ

โœ… ๋ณ€๊ฒฝ๋กœ์ง

  • totalschedule ํŒจํ‚ค์ง€ ๋ฐ ์„ธ๋ถ€ ํŒจํ‚ค์ง€ ์ƒ์„ฑํ–ˆ์Šต๋‹ˆ๋‹ค.
  • publicScheduleAdmin์— ์žˆ๋˜ ํ•จ์ˆ˜๋“ค์„ totalScheduleController๋กœ ์ด๋™ํ–ˆ์Šต๋‹ˆ๋‹ค.
  • admin๊ด€๋ จ securityConfigํŒŒ์ผ ์ˆ˜์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

๐Ÿ’กIssue ๋ฒˆํ˜ธ

@codecov

codecov Bot commented Dec 31, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests โœ…

Project coverage is 76.99%. Comparing base (a71cfc0) to head (21ac71b).
Report is 16 commits behind head on dev.

Additional details and impacted files
@@             Coverage Diff              @@
##                dev    #1093      +/-   ##
============================================
+ Coverage     74.42%   76.99%   +2.57%     
- Complexity     1227     1691     +464     
============================================
  Files           399      574     +175     
  Lines          5524     7780    +2256     
  Branches        440      525      +85     
============================================
+ Hits           4111     5990    +1879     
- Misses         1220     1547     +327     
- Partials        193      243      +50     
Flag Coverage ฮ”
integrationTest 72.59% <100.00%> (+3.20%) โฌ†๏ธ
unitTest 30.11% <0.00%> (-5.86%) โฌ‡๏ธ

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

โ˜” View full report in Codecov by Sentry.
๐Ÿ“ข Have feedback on the report? Share it here.

wonies
wonies previously approved these changes Dec 31, 2024

@wonies wonies left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

๋กœ์ง ๋ฐ”๊พธ์‹œ๋А๋ผ ๊ณ ์ƒ๋งŽ์œผ์…จ์Šต๋‹ˆ๋‹ค!

private MockMvc mockMvc;

@Autowired
EntityManager em;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

em์„ ์“ฐ๋Š” ๋ถ€๋ถ„์ด ์—†๋Š”๋ฐ, ์ด๊ฑด ๊ทธ๋ƒฅ ์„ ์–ธํ•ด๋„ ๋˜๋Š” ๊ฑด๊ฐ€์š”? ๊ถ๊ธˆํ•ด์„œ ์—ฌ์ญค๋ด…๋‹ˆ๋‹ท

Page<PublicSchedule> publicSchedules = publicScheduleAdminRepository.findAllByClassification(
detailClassification, pageable);

List<TotalScheduleAdminResDto> publicScheduleList = publicSchedules.stream()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

์ž๋ฐ” 11๋ฒ„์ „ collects

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants