Re: [Minios-devel] [UNIKRAFT/RUBY PATCH 5/8] tests: Build tests

[Stefan Teodorescu <stefanl.teodorescu@xxxxxxxxx>]

There is a typo when initializing the file test, you should change it
to Init_file, instead of Init_time.

Reviewed-by: Stefan Teodorescu <stefanl.teodorescu@xxxxxxxxx>

On Thu, Dec 12, 2019 at 7:42 PM Costin Lupu <costin.lupu@xxxxxxxxx> wrote:
>
> The Ruby unit testing depends on both extensions and encodings. We currently
> add support for running only the time and file tests. Enabling the rest of the
> tests is subject of further work.
>
> Signed-off-by: Costin Lupu <costin.lupu@xxxxxxxxx>
> ---
>  Config.uk               |  7 +++++++
>  Makefile.uk             | 42 +++++++++++++++++++++++++++++++++++++++++
>  generated/ext/extinit.c |  6 ++++++
>  3 files changed, 55 insertions(+)
>
> diff --git a/Config.uk b/Config.uk
> index a494540..48b9669 100644
> --- a/Config.uk
> +++ b/Config.uk
> @@ -131,4 +131,11 @@ config LIBRUBY_ENC
>         depends on !LIBRUBY_MINI
>         default y
>
> +config LIBRUBY_TEST
> +       bool "Enable unit testing"
> +       depends on !LIBRUBY_MINI
> +       depends on LIBRUBY_EXT
> +       depends on LIBRUBY_ENC
> +       default y
> +
>  endif
> diff --git a/Makefile.uk b/Makefile.uk
> index 041e7ec..9ea83a3 100644
> --- a/Makefile.uk
> +++ b/Makefile.uk
> @@ -41,6 +41,7 @@
>  $(eval $(call addlib_s,libruby,$(CONFIG_LIBRUBY)))
>  $(eval $(call addlib_s,libruby_ext,$(CONFIG_LIBRUBY_EXT)))
>  $(eval $(call addlib_s,libruby_enc,$(CONFIG_LIBRUBY_ENC)))
> +$(eval $(call addlib_s,libruby_test,$(CONFIG_LIBRUBY_TEST)))
>
>  
> ################################################################################
>  # Sources
> @@ -544,6 +545,31 @@ LIBRUBY_ENC_SRCS-y += 
> $(LIBRUBY_SRC)/enc/trans/single_byte.c
>  LIBRUBY_ENC_SRCS-y += $(LIBRUBY_SRC)/enc/trans/utf8_mac.c
>  LIBRUBY_ENC_SRCS-y += $(LIBRUBY_SRC)/enc/trans/utf_16_32.c
>
> +################################################################################
> +# Unit testing
> +################################################################################
> +LIBRUBY_TEST_CFLAGS-y += $(LIBRUBY_CINCLUDES-y)
> +LIBRUBY_TEST_CFLAGS-y += $(LIBRUBY_SUPPRESS_FLAGS)
> +LIBRUBY_TEST_CFLAGS-y += -DRUBY_EXTCONF_H=\"extconf.h\"
> +
> +$(LIBRUBY_SRC)/ext/-test-/file/init_file.c: 
> $(LIBRUBY_SRC)/ext/-test-/file/init.c
> +       ln -s $< $@
> +LIBRUBY_TEST_SRCS-y += $(LIBRUBY_SRC)/ext/-test-/file/init_file.c
> +LIBRUBY_TEST_INIT-FILE_FLAGS-y += -I$(LIBRUBY_SRC)/ext/-test-/file
> +LIBRUBY_TEST_SRCS-y += $(LIBRUBY_SRC)/ext/-test-/file/fs.c
> +LIBRUBY_TEST_FS_FLAGS-y += -I$(LIBRUBY_SRC)/ext/-test-/file
> +LIBRUBY_TEST_SRCS-y += $(LIBRUBY_SRC)/ext/-test-/file/stat.c
> +LIBRUBY_TEST_STAT_FLAGS-y += -I$(LIBRUBY_SRC)/ext/-test-/file
> +
> +$(LIBRUBY_SRC)/ext/-test-/time/init_time.c: 
> $(LIBRUBY_SRC)/ext/-test-/time/init.c
> +       ln -s $< $@
> +LIBRUBY_TEST_SRCS-y += $(LIBRUBY_SRC)/ext/-test-/time/init_time.c
> +LIBRUBY_TEST_INIT-TIME_FLAGS-y += -I$(LIBRUBY_SRC)/ext/-test-/time
> +LIBRUBY_TEST_SRCS-y += $(LIBRUBY_SRC)/ext/-test-/time/leap_second.c
> +LIBRUBY_TEST_LEAP_SECOND_FLAGS-y += -I$(LIBRUBY_SRC)/ext/-test-/time
> +LIBRUBY_TEST_SRCS-y += $(LIBRUBY_SRC)/ext/-test-/time/new.c
> +LIBRUBY_TEST_NEW_FLAGS-y += -I$(LIBRUBY_SRC)/ext/-test-/time
> +
>  
> ################################################################################
>  # Prepare rules
>  
> ################################################################################
> @@ -564,3 +590,19 @@ $(LIBRUBY_BUILD)/.build: $(LIBRUBY_BUILD)/.configured
>
>  UK_PREPARE += $(LIBRUBY_BUILD)/.build
>
> +ifeq ($(CONFIG_LIBRUBY_TEST),y)
> +# Customize config: configure stack size
> +$(APP_BASE)/.config.orig: $(APP_BASE)/.config
> +       $(call verbose_cmd,CONFIG,libruby: $(notdir $@), \
> +               cp $(APP_BASE)/.config $@ && \
> +               sed -i 
> 's/^CONFIG_STACK_SIZE_PAGE_ORDER=.*$$/CONFIG_STACK_SIZE_PAGE_ORDER=10/g' 
> $(APP_BASE)/.config)
> +
> +# We need a bigger stack for tests
> +LIBRUBY_TEST_PREPARED_DEPS = \
> +       $(APP_BASE)/.config.orig \
> +
> +$(LIBRUBY_TEST_BUILD)/.prepared: $(LIBRUBY_TEST_PREPARED_DEPS)
> +
> +UK_PREPARE += $(LIBRUBY_TEST_BUILD)/.prepared
> +endif
> +
> diff --git a/generated/ext/extinit.c b/generated/ext/extinit.c
> index dcf9631..57134c5 100644
> --- a/generated/ext/extinit.c
> +++ b/generated/ext/extinit.c
> @@ -102,4 +102,10 @@ void Init_ext(void)
>  #if CONFIG_LIBRUBY_EXT_SDBM
>      init(Init_sdbm, "sdbm");
>  #endif
> +
> +#if CONFIG_LIBRUBY_TEST
> +    /* TODO add more tests */
> +    init(Init_time, "-test-/file");
> +    init(Init_time, "-test-/time");
> +#endif
>  }
> --
> 2.20.1
>

_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel