Skip to content

Commit

Permalink
disable tests for missing aesthetics
Browse files Browse the repository at this point in the history
These have been made redundant. At some point in the future we may
wish to enable such a thing again, but that requires a bit of thought
where such an exception would then be thrown
  • Loading branch information
Vindaar committed Feb 9, 2024
1 parent 447d0e5 commit 09c3c3c
Showing 1 changed file with 19 additions and 19 deletions.
38 changes: 19 additions & 19 deletions tests/tests.nim
Original file line number Diff line number Diff line change
Expand Up @@ -386,25 +386,25 @@ suite "GgPlot":
epsilon = 1e-6)
plt.ggdraw("exp2.pdf")

test "Plot requiring `y` scale throws if none given":
let mpg = readCsv("data/mpg.csv")
try:
ggplot(mpg, aes("year")) +
geom_point() +
ggsave("never_produced.png")
check false
except AestheticError:
check true

test "Plot not needing `y` scale throws if given":
let mpg = readCsv("data/mpg.csv")
try:
ggplot(mpg, aes(x = "class", y = "cyl")) +
geom_bar() +
ggsave("never_produced.png")
check false
except AestheticError:
check true
#test "Plot requiring `y` scale throws if none given":
# let mpg = readCsv("data/mpg.csv")
# try:
# ggplot(mpg, aes("year")) +
# geom_point() +
# ggsave("never_produced.png")
# check false
# except AestheticError:
# check true
#
#test "Plot not needing `y` scale throws if given":
# let mpg = readCsv("data/mpg.csv")
# try:
# ggplot(mpg, aes(x = "class", y = "cyl")) +
# geom_bar() +
# ggsave("never_produced.png")
# check false
# except AestheticError:
# check true

suite "Theme":
test "Canvas background color":
Expand Down

0 comments on commit 09c3c3c

Please sign in to comment.