Update documentation and add compiling script docs

- Clean up contract compiler script

Author: fselmo

docs/contributing.rst

@@ -195,18 +195,16 @@ Within the ``pytest`` scope, :file:`conftest.py` files are used for common code
 shared between modules that exist within the same directory as that particular
 :file:`conftest.py` file.
 
-Unit Testing
-^^^^^^^^^^^^
+Unit Testing and eth-tester Tests
+^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
-Unit tests are meant to test the logic of smaller chunks (or units) of the
-codebase without having to be wired up to a client. Most of the time this
-means testing selected methods on their own. They are meant to test the logic
-of your code and make sure that you get expected outputs out of selected inputs.
+Our unit tests are grouped together with tests against the ``eth-tester`` library,
+using the ``py-evm`` library as a backend, via the ``EthereumTesterProvider``.
 
-Our unit tests live under appropriately named child directories within the
-``/tests`` directory. The core of the unit tests live under ``/tests/core``.
-Do your best to follow the existing structure when choosing where to add
-your unit test.
+These tests live under appropriately named child directories within the
+``/tests`` directory. The core of these tests live under ``/tests/core``.
+Do your best to follow the existing structure when adding a test and make sure
+that its location makes sense.
 
 Integration Testing
 ^^^^^^^^^^^^^^^^^^^
@@ -217,25 +215,76 @@ confused with pytest fixtures). These zip file fixtures, which also live in the
 ``/tests/integration`` directory, are configured to run the specific client we are
 testing against along with a genesis configuration that gives our tests some
 pre-determined useful objects (like unlocked, pre-loaded accounts) to be able to
-interact with the client and run our tests.
+interact with the client when we run our tests.
 
-Though the setup lives in ``/tests/integration``, our integration module tests are
-written across different files within ``/web3/_utils/module_testing``. The tests
-are written there but run configurations exist across the different files within
-``/tests/integration/``. The parent ``/integration`` directory houses some common
-configuration shared across all client tests, whereas the ``/go_ethereum`` directory
-houses common code to be shared among respective client tests.
+The parent ``/integration`` directory houses some common configuration shared across
+all client tests, whereas the ``/go_ethereum`` directory houses common code to be
+shared across geth-specific provider tests. Though the setup run configurations exist
+across the different files within ``/tests/integration``, our integration module tests
+are written across different files within ``/web3/_utils/module_testing``.
 
 * :file:`common.py` files within the client directories contain code that is shared across
   all provider tests (http, ipc, and ws). This is mostly used to override tests that span
   across all providers.
-* :file:`conftest.py` files within each of these directories contain mostly code that can
-  be *used* by all test files that exist within the same directory as the :file:`conftest.py`
-  file. This is mostly used to house pytest fixtures to be shared among our tests. Refer to
-  the `pytest documentation on fixtures`_ for more information.
-* :file:`test_{client}_{provider}.py` (e.g. :file:`test_goethereum_http.py`) files are where
-  client-and-provider-specific test configurations exist. This is mostly used to override tests
-  specific to the provider type for the respective client.
+* :file:`conftest.py` files within each of these directories contain mostly code that
+  can be *used* by all test files that exist within the same directory or subdirectories
+  of the :file:`conftest.py` file. This is mostly used to house pytest fixtures to be
+  shared among our tests. Refer to the `pytest documentation on fixtures`_ for more
+  information.
+* ``test_{client}_{provider}.py`` files (e.g. :file:`test_goethereum_http.py`) are where
+  client-and-provider-specific test configurations exist. This is mostly used to
+  override tests specific to the provider type for the respective client.
+
+
+Working With Test Contracts
+^^^^^^^^^^^^^^^^^^^^^^^^^^^
+
+Contracts used for testing exist under ``web3/_utils/contract_sources``. These contracts
+get compiled via the ``compile_contracts.py`` script in the same directory. To use
+this script, simply pass the Solidity version to be used to compile the contracts as an
+argument at the command line.
+
+Arguments for the script are:
+    -v or --version         Solidity version to be used to compile the contracts. If
+                            blank, the script uses the latest hard-coded version
+                            specified within the script.
+
+    -f or --filename        If left blank, all .sol files will be compiled and the
+                            respective contract data will be generated. Pass in a
+                            specific ``.sol`` filename here to compile just one file.
+
+
+To run the script, you will need the ``py-solc-x`` library for compiling the files
+as well as ``black`` for linting. You can install those independently or install the
+full ``[dev]`` package extra as shown below.
+
+.. code:: sh
+
+    $ pip install "web3[dev]"
+
+The following example compiles all the contracts and generates their respective
+contract data that is used across our test files for the test suites. This data gets
+generated within the ``contract_data`` subdirectory within the ``contract_sources``
+folder.
+
+.. code-block:: bash
+
+    $ cd ../web3.py/web3/_utils/contract_sources
+    $ python compile_contracts.py -v 0.8.17
+    Compiling OffchainLookup
+    ...
+    ...
+    reformatted ...
+
+To compile and generate contract data for only one ``.sol`` file, specify using the
+filename with the ``-f`` (or ``--filename``) argument flag.
+
+.. code-block:: bash
+
+    $ cd ../web3.py/web3/_utils/contract_sources
+    $ python compile_contracts.py -v 0.8.17 -f OffchainLookup.sol
+    Compiling OffchainLookup.sol
+    reformatted ...
 
 
 Manual Testing
@@ -246,7 +295,7 @@ you can install it from your development directory:
 
 .. code:: sh
 
-   $ pip install -e ../path/to/web3py
+   $ pip install -e ../path/to/web3py
 
 
 Documentation

newsfragments/2797.internal.rst

@@ -0,0 +1,2 @@
+Re-compile all test contracts with latest Solidity version. Refactor test fixtures. Adds a script that compiles all test contracts to the same directory with selected Solidity version.
+

web3/_utils/contract_sources/compile_contracts.py

@@ -1,37 +1,93 @@
+"""
+Arguments for the script are:
+    -v or --version         Solidity version to be used to compile the contracts. If
+                            blank, the script uses the latest hard-coded version
+                            specified within the script.
+
+    -f or --filename        If left blank, all .sol files will be compiled and the
+                            respective contract data will be generated. Pass in a
+                            specific ``.sol`` filename here to compile just one file.
+
+
+To run the script, you will need the ``py-solc-x`` library for compiling the files
+as well as ``black`` for linting. You can install those independently or install the
+full ``[dev]`` package extra as shown below.
+
+.. code:: sh
+
+    $ pip install "web3[dev]"
+
+The following example compiles all the contracts and generates their respective
+contract data that is used across our test files for the test suites. This data gets
+generated within the ``contract_data`` subdirectory within the ``contract_sources``
+folder.
+
+.. code-block:: bash
+
+    $ cd ../web3.py/web3/_utils/contract_sources
+    $ python compile_contracts.py -v 0.8.17
+    Compiling OffchainLookup
+    ...
+    ...
+    reformatted ...
+
+To compile and generate contract data for only one ``.sol`` file, specify using the
+filename with the ``-f`` (or ``--filename``) argument flag.
+
+.. code-block:: bash
+
+    $ cd ../web3.py/web3/_utils/contract_sources
+    $ python compile_contracts.py -v 0.8.17 -f OffchainLookup.sol
+    Compiling OffchainLookup.sol
+    reformatted ...
+"""
+
+
+import argparse
 import os
 import re
-import sys
 from typing import (
     Any,
 )
 
 import solcx
 
-SOLIDITY_VERSION_REGEX = re.compile(r"\d*\.\d+")  # 0.0.0 pattern
-CONFIGURED_SOLIDITY_VERSION = "0.8.17"
-
-user_provided_version = sys.argv[-1]
-
-solidity_version = (
-    user_provided_version
-    if SOLIDITY_VERSION_REGEX.match(user_provided_version)
-    else CONFIGURED_SOLIDITY_VERSION
+arg_parser = argparse.ArgumentParser()
+arg_parser.add_argument(
+    "-v", "--version", help="Solidity version for compiling contracts."
+)
+arg_parser.add_argument(
+    "-f",
+    "--filename",
+    help="(optional) The filename if only one file is to be compiled - "
+    "otherwise all .sol files will be compiled at once.",
 )
+user_args = arg_parser.parse_args()
 
+CONFIGURED_SOLIDITY_VERSION = "0.8.17"
+# establish Solidity version from user-provided arg or use hard-coded version
+user_sol_version = user_args.version
 
+solidity_version = user_sol_version if user_sol_version else CONFIGURED_SOLIDITY_VERSION
 solcx.install_solc(solidity_version)
 solcx.set_solc_version(solidity_version)
 
 
-def compile_dot_sol_files(dot_sol_filename: str) -> dict[str, Any]:
+# compile just the .sol file specified or all .sol files
+all_dot_sol_files = [f for f in os.listdir(os.getcwd()) if f.endswith(".sol")]
+user_filename = user_args.filename
+files_to_compile = [user_filename] if user_filename else all_dot_sol_files
+
+
+def _compile_dot_sol_files(dot_sol_filename: str) -> dict[str, Any]:
     compiled = solcx.compile_files(
         [f"./{dot_sol_filename}"],
         output_values=["abi", "bin", "bin-runtime"],
     )
     return compiled
 
 
-def get_compiled_contract_data(
+def _get_compiled_contract_data(
     sol_file_output: dict[str, dict[str, str]],
     dot_sol_filename: str,
     contract_name: str = None,
@@ -54,8 +110,9 @@ def get_compiled_contract_data(
 
 contracts_in_file = {}
 
-for filename in os.listdir(os.getcwd()):
-    if ".sol" in filename:
+
+def compile_files(file_list: list[str]) -> None:
+    for filename in file_list:
         with open(os.path.join(os.getcwd(), filename), "r") as f:
             dot_sol_file = f.readlines()
 
@@ -70,54 +127,54 @@ def get_compiled_contract_data(
 
         contracts_in_file[filename] = contract_names
 
-
-for dot_sol_filename in contracts_in_file.keys():
-    filename_no_extension = dot_sol_filename.replace(".sol", "")
-    split_and_lowercase = [
-        i.lower() for i in re.split(r"([A-Z][a-z]*)", filename_no_extension) if i
-    ]
-    python_filename = f"{'_'.join(split_and_lowercase)}.py"
-    python_file_path = os.path.join(os.getcwd(), "contract_data", python_filename)
-
-    try:
-        # clean up existing files
-        os.remove(python_file_path)
-    except FileNotFoundError:
-        pass
-
-    print(f"compiling {dot_sol_filename}")
-    compiled_dot_sol_data = compile_dot_sol_files(dot_sol_filename)
-
-    with open(python_file_path, "w") as f:
-        f.write(f'"""\nGenerated by `{os.path.basename(__file__)}` script.\n')
-        f.write(f'Compiled with Solidity v{solidity_version}.\n"""\n\n')
-
-        for c in contracts_in_file[dot_sol_filename]:
-            c_name_split_and_uppercase = [
-                i.upper() for i in re.split(r"([A-Z0-9][a-z0-9]*)", c) if i
-            ]
-            contract_upper = "_".join(c_name_split_and_uppercase)
-
-            c_data = get_compiled_contract_data(
-                compiled_dot_sol_data, dot_sol_filename, c
-            )
-
-            contract_source = (
-                f"# source: web3/_utils/contract_sources/{dot_sol_filename}:{c}"
-            )
-            if len(contract_source) > 88:
-                contract_source += "  # noqa: E501"
-
-            f.write(f"{contract_source}\n")
-            f.write(f'{contract_upper}_BYTECODE = "{c_data["bin"]}"  # noqa: E501\n')
-            f.write(
-                f'{contract_upper}_RUNTIME = "{c_data["bin-runtime"]}"  # noqa: E501\n'
-            )
-            f.write(f"{contract_upper}_ABI = {c_data['abi']}\n")
-            f.write(contract_upper + "_DATA = {\n")
-            f.write(f'    "bytecode": {contract_upper}_BYTECODE,\n')
-            f.write(f'    "bytecode_runtime": {contract_upper}_RUNTIME,\n')
-            f.write(f'    "abi": {contract_upper}_ABI,\n')
-            f.write("}\n\n\n")
-
+    for dot_sol_filename in contracts_in_file.keys():
+        filename_no_extension = dot_sol_filename.replace(".sol", "")
+        split_and_lowercase = [
+            i.lower() for i in re.split(r"([A-Z][a-z]*)", filename_no_extension) if i
+        ]
+        python_filename = f"{'_'.join(split_and_lowercase)}.py"
+        python_file_path = os.path.join(os.getcwd(), "contract_data", python_filename)
+        try:
+            # clean up existing files
+            os.remove(python_file_path)
+        except FileNotFoundError:
+            pass
+        print(f"compiling {dot_sol_filename}")
+        compiled_dot_sol_data = _compile_dot_sol_files(dot_sol_filename)
+        with open(python_file_path, "w") as f:
+            f.write(f'"""\nGenerated by `{os.path.basename(__file__)}` script.\n')
+            f.write(f'Compiled with Solidity v{solidity_version}.\n"""\n\n')
+
+            for c in contracts_in_file[dot_sol_filename]:
+                c_name_split_and_uppercase = [
+                    i.upper() for i in re.split(r"([A-Z0-9][a-z0-9]*)", c) if i
+                ]
+                contract_upper = "_".join(c_name_split_and_uppercase)
+
+                c_data = _get_compiled_contract_data(
+                    compiled_dot_sol_data, dot_sol_filename, c
+                )
+
+                contract_source = (
+                    f"# source: web3/_utils/contract_sources/{dot_sol_filename}:{c}"
+                )
+                if len(contract_source) > 88:
+                    contract_source += "  # noqa: E501"
+
+                f.write(f"{contract_source}\n")
+                f.write(
+                    f'{contract_upper}_BYTECODE = "{c_data["bin"]}"  # noqa: E501\n'
+                )
+                f.write(
+                    f'{contract_upper}_RUNTIME = "{c_data["bin-runtime"]}"  # noqa: E501\n'
+                )
+                f.write(f"{contract_upper}_ABI = {c_data['abi']}\n")
+                f.write(contract_upper + "_DATA = {\n")
+                f.write(f'    "bytecode": {contract_upper}_BYTECODE,\n')
+                f.write(f'    "bytecode_runtime": {contract_upper}_RUNTIME,\n')
+                f.write(f'    "abi": {contract_upper}_ABI,\n')
+                f.write("}\n\n\n")
+
+
+compile_files(files_to_compile)
 os.system(f"black {os.getcwd()}")