Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: 블로그 방문자 수, 포스트 조회 수 통계 작업 오류 수정 #138

Merged
merged 2 commits into from
Dec 7, 2023
Merged

fix: 블로그 방문자 수, 포스트 조회 수 통계 작업 오류 수정 #138

merged 2 commits into from
Dec 7, 2023

Conversation

shin-mallang
Copy link
Member

@shin-mallang shin-mallang linked an issue Dec 7, 2023 that may be closed by this pull request
[🐞Bug] 블로그 방문자, 포스트 조회 수 통계 작업 오류 수정 #136
Closed
@shin-mallang shin-mallang self-assigned this Dec 7, 2023
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Dec 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 7, 2023

Test Results

761 tests  +4   760 ✔️ +4   27s ⏱️ -1s
284 suites ±0       1 💤 ±0 
284 files   ±0       0 ❌ ±0 

Results for commit 13254ee. ± Comparison against base commit db36561.

This pull request removes 19 and adds 10 tests. Note that renamed tests count towards both. 

 
          Shin._.Mallang
        title-value
    </title>
    <meta property="og:title" content="Shin._.Mallang"/>
    <title>
    HI
 </head>
</body>
…

com.mallang.reference.domain.ReferenceLinkTest$생성_시 ‑ [3] title= 
 
com.mallang.reference.domain.ReferenceLinkTitleTest$생성_시 ‑ [4] title=

com.mallang.reference.infrastructure.jsoup.JsoupUrlTitleMetaInfoFetcherTest ‑ [1] html=<!DOCTYPE html>
<head>
    <title>
          Shin._.Mallang
    </title>
 </head>

<body>
    HI
</body>

</html>

com.mallang.reference.infrastructure.jsoup.JsoupUrlTitleMetaInfoFetcherTest ‑ [2] html=<!DOCTYPE html>
<head>
    <title>
        title-value
    </title>
    <meta property="og:title" content="Shin._.Mallang"/>
 </head>

<body>
    HI
</body>

</html>

com.mallang.statistics.batch.BlogVisitStatisticJobTest ‑ 시작시간은 포함되며 끝시간은 포함되지 않는다
com.mallang.statistics.batch.BlogVisitStatisticJobTest ‑ 특정 시간대에 포함되지 않으면 집계하지 않는다
com.mallang.statistics.batch.BlogVisitStatisticJobTest ‑ 특정 시간대의 집계되지 않은 모든 조회 이력을 가져와 블로그별로 그리고 일자별로 개수를 집계한다
com.mallang.statistics.batch.PostViewStatisticJobTest ‑ 시작시간은 포함되며 끝시간은 포함되지 않는다
com.mallang.statistics.batch.PostViewStatisticJobTest ‑ 특정 시간대에 포함되지 않으면 집계하지 않는다
com.mallang.statistics.batch.PostViewStatisticJobTest ‑ 특정 시간대의 집계되지 않은 모든 조회 이력을 가져와 블로그별로 그리고 일자별로 개수를 집계한다

@shin-mallang shin-mallang merged commit 8b6ddb5 into main Dec 7, 2023
5 checks passed
shin-mallang added a commit that referenced this pull request Dec 10, 2023
@shin-mallang
fix: 블로그 방문자 수, 포스트 조회 수 통계 작업 오류 수정 (#138)
a170185 
* [#136] feat: 블로그 방문자 수 통계 작업 시 시작 시간대와 끝 시간대를 설정할 수 있도록 변경

* [#136] feat: 포스트 조회수 통계 작업 시 시작 시간대와 끝 시간대를 설정할 수 있도록 변경

(cherry picked from commit 8b6ddb5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@shin-mallang shin-mallang

Labels
🎉 오류해결
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[🐞Bug] 블로그 방문자, 포스트 조회 수 통계 작업 오류 수정
1 participant
@shin-mallang