You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by dk...@apache.org on 2018/10/23 14:04:16 UTC
[sling-org-apache-sling-app-cms] branch master updated: Zapping
gremlins (e.g. non-ASCII characters which were causing some slight (but
enough) differences in the word counts)
This is an automated email from the ASF dual-hosted git repository.
dklco pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-app-cms.git
The following commit(s) were added to refs/heads/master by this push:
new fc3d098 Zapping gremlins (e.g. non-ASCII characters which were causing some slight (but enough) differences in the word counts)
fc3d098 is described below
commit fc3d09824f3d0a0d1f5ae031a9c71b0e287f846a
Author: Dan Klco <dk...@apache.org>
AuthorDate: Tue Oct 23 10:04:07 2018 -0400
Zapping gremlins (e.g. non-ASCII characters which were causing some
slight (but enough) differences in the word counts)
---
.../cms/core/readability/impl/ReadabilityServiceImplTest.java | 7 ++++---
core/src/test/resources/source3.txt | 4 ++--
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/core/src/test/java/org/apache/sling/cms/core/readability/impl/ReadabilityServiceImplTest.java b/core/src/test/java/org/apache/sling/cms/core/readability/impl/ReadabilityServiceImplTest.java
index 7baa2d6..709dcb4 100644
--- a/core/src/test/java/org/apache/sling/cms/core/readability/impl/ReadabilityServiceImplTest.java
+++ b/core/src/test/java/org/apache/sling/cms/core/readability/impl/ReadabilityServiceImplTest.java
@@ -111,6 +111,7 @@ public class ReadabilityServiceImplTest {
log.debug("ARI {}", ari);
assertTrue(ari > 16 && ari < 18);
+
ari = readabilityService.calculateARI(source2);
@@ -122,7 +123,7 @@ public class ReadabilityServiceImplTest {
log.debug("ARI {}", ari);
- assertTrue(ari > 5 && ari < 6);
+ assertTrue(ari > 6 && ari < 7);
log.info("Tests successful!");
}
@@ -147,7 +148,7 @@ public class ReadabilityServiceImplTest {
log.debug("Average Grade Level {}", averageGradeLevel);
- assertTrue(averageGradeLevel > 5 && averageGradeLevel < 6);
+ assertTrue(averageGradeLevel > 6 && averageGradeLevel < 7);
log.info("Tests successful!");
}
@@ -205,7 +206,7 @@ public class ReadabilityServiceImplTest {
assertNotNull(text);
assertEquals(11, text.getSentences().size());
- assertEquals(159, text.getWordCount());
+ assertEquals(157, text.getWordCount());
assertEquals(7, text.getComplexWordCount());
log.info("Tests successful!");
diff --git a/core/src/test/resources/source3.txt b/core/src/test/resources/source3.txt
index 0643da9..2f30645 100644
--- a/core/src/test/resources/source3.txt
+++ b/core/src/test/resources/source3.txt
@@ -21,8 +21,8 @@ The 37-year-old actor confessed that his two young daughters with Eva Mendes are
Gosling said on "Jimmy Kimmel Live!" Wednesday that the two girls visited him on the set of "First Man" and saw him in the full astronaut gig. It made an impression.
-“They came to set and they came on this day where I was shooting a launch scene,” Gosling told Kimmel. “So I was in the space suit and pretending to launch. I think they think I’m an astronaut now.”
+"They came to set and they came on this day where I was shooting a launch scene," Gosling told Kimmel. "So I was in the space suit and pretending to launch. I think they think I'm an astronaut now."
Esmeralda, 4, even took it a step further when discussing her father's supposed workplace.
-“My oldest said the other day, she pointed to the moon and she was like, ‘That’s where you work, right?'” Gosling said.
\ No newline at end of file
+"My oldest said the other day, she pointed to the moon and she was like, 'That's where you work, right?'" Gosling said.
\ No newline at end of file