Add GH Actions which will build Dockerfile if updated

[antonbabenko]

GH Actions for pull-requests which runs docker build when Dockerfile changes and run the smoke test?

No need to push it to GHCR but just getting versions should be enough. Similar to this one - https://github.com/antonbabenko/pre-commit-terraform/blob/master/.github/workflows/build-image.yaml

Originally posted by @antonbabenko in #288 (comment)

[mohitsaxenaknoldus]

Hey, can I work on this?

[antonbabenko]

Sure, @mohitsaxenaknoldus !

[mohitsaxenaknoldus]

Where can I find the smoke tests?

[MaxymVlasov]

If you write them and populate them somewhere in the tests/ folder + info on how to use them in .github/CONTRIBUTING.md it will be helpful.

For now, we have only some kind of performance tests

[mohitsaxenaknoldus]

So I also have to create the smoke tests as well then?

[MaxymVlasov]

Yep

[mohitsaxenaknoldus]

I've added workflow which will build the Dockerfile if it was changed but I've never worked with smoke tests before :(

[mohitsaxenaknoldus]

Here's the work so far: #318

[MaxymVlasov]

Maybe we already have the simplest optimistic smoke test here:

pre-commit-terraform/Dockerfile

Lines 129 to 141 in c5f2a61

	# Checking binaries versions and write it to debug file
	RUN . /.env && \
	F=tools_versions_info && \
	pre-commit --version >> $F && \
	./terraform --version | head -n 1 >> $F && \
	(if [ "$CHECKOV_VERSION" != "false" ]; then echo "checkov $(checkov --version)" >> $F; else echo "checkov SKIPPED" >> $F ; fi) && \
	(if [ "$INFRACOST_VERSION" != "false" ]; then echo "$(./infracost --version)" >> $F; else echo "infracost SKIPPED" >> $F ; fi) && \
	(if [ "$TERRAFORM_DOCS_VERSION" != "false" ]; then ./terraform-docs --version >> $F; else echo "terraform-docs SKIPPED" >> $F ; fi) && \
	(if [ "$TERRAGRUNT_VERSION" != "false" ]; then ./terragrunt --version >> $F; else echo "terragrunt SKIPPED" >> $F ; fi) && \
	(if [ "$TERRASCAN_VERSION" != "false" ]; then echo "terrascan $(./terrascan version)" >> $F; else echo "terrascan SKIPPED" >> $F ; fi) && \
	(if [ "$TFLINT_VERSION" != "false" ]; then ./tflint --version >> $F; else echo "tflint SKIPPED" >> $F ; fi) && \
	(if [ "$TFSEC_VERSION" != "false" ]; then echo "tfsec $(./tfsec --version)" >> $F; else echo "tfsec SKIPPED" >> $F ; fi) && \
	echo -e "\n\n" && cat $F && echo -e "\n\n"

@antonbabenko do you'd like to have pre-commit run on some predefined code for each hook? Or what do you mean by "smoke test"?

[antonbabenko]

I think the simplest optimistic smote test you refer to is a pretty accurate definition of what I meant by "smoke test" :)

[antonbabenko]

This issue has been resolved in version 1.63.0 🎉