378 Commits

Author	SHA1	Message	Date
Angus Gratton	f4ab9d9247	extmod/vfs_blockdev: Implement common helper for read and write. ... - Code size saving as all of these functions are very similar. - Resolves the "TODO" of the plain read and write functions not propagating errors. An error in the underlying block device now causes VFatFs to return EIO, for example. This work was funded through GitHub Sponsors. Signed-off-by: Angus Gratton <angus@redyak.com.au>	2024-09-26 22:08:48 +10:00
Angus Gratton	4f6d4b2b49	extmod/vfs_blockdev: Check block device function positive results. ... A positive result here can result in eventual memory corruption as littlefs expects the result of a cache read/write function to be 0 or a negative integer for an error. Closes #13046 This work was funded through GitHub Sponsors. Signed-off-by: Angus Gratton <angus@redyak.com.au>	2024-09-26 22:08:48 +10:00
robert-hh	e23fdb1f77	tests/extmod/machine_uart_irq_txidle.py: Simplify the test script. ... Now that no minimal delay time is required for SAMD devices. Signed-off-by: robert-hh <robert@hammelrath.com>	2024-09-06 17:43:49 +10:00
Damien George	b8513e6137	tests/extmod: Add test for machine.UART.IRQ_TXIDLE. ... The test checks whether the message created by the IRQ handler appears about at the end of the data sent by UART. Supported MCUs resp. boards: - RP2040 - Teensy 4.x - Adafruit ItsyBitsy M0 - Adafruit ItsyBitsy M4 - NRF52 (Arduino Nano Connect 33 BLE) Signed-off-by: Damien George <damien@micropython.org>	2024-08-29 16:48:09 +10:00
Angus Gratton	fb4ae1eeec	test/extmod: Fix machine_spi_rate test on ESP32-C3. ... Update to the test added in 1e98c4cb75, changes the SPI pins for ESP32-C3 (IO 18 and 19 are the native USB pins). Signed-off-by: Angus Gratton <angus@redyak.com.au>	2024-08-14 15:03:27 +10:00
Peter Züger	1473ed4c64	tests/extmod/ssl_keycert.py: Add test for PKCS8 formatted DER key. ... Signed-off-by: Peter Züger <zueger.peter@icloud.com>	2024-08-13 17:32:32 +10:00
Angus Gratton	1e98c4cb75	tests/extmod: Add machine_spi_rate test. ... Based on machine_i2s_rate, allows testing basic SPI functionality and timings. Implemented and confirmed working for rp2, esp32, and pyboard. This work was funded through GitHub Sponsors. Signed-off-by: Angus Gratton <angus@redyak.com.au>	2024-08-07 15:07:13 +10:00
Damien George	a4b3825bd4	tests/extmod: Rename machine_timer exp file to machine_soft_timer. ... This was missed in 9ba04cc756 Signed-off-by: Damien George <damien@micropython.org>	2024-08-07 15:02:52 +10:00
Angus Gratton	6d05424754	tests/extmod: Add esp32 support to the machine_i2s_rate test. ... This work was funded through GitHub Sponsors. Signed-off-by: Angus Gratton <angus@redyak.com.au>	2024-08-02 10:31:19 +10:00
Angus Gratton	9ba04cc756	tests/extmod: Skip soft machine.Timer test on esp32 port. ... Also rename the test to reflect that it's a soft timer test. Signed-off-by: Angus Gratton <angus@redyak.com.au>	2024-08-02 10:27:53 +10:00
stijn	444d7bacbe	extmod/moductypes: Validate the descriptor tuple. ... Fixes various null dereferencing, out-of-bounds memory accesses and `assert(0)` failures in the case of an invalid `uctypes` descriptor. By design `uctypes` can crash because it accesses arbitrary memory, but at least describing the descriptor layout should be forced to be correct and not crash. Fixes issue #12702. Signed-off-by: stijn <stijn@ignitron.net>	2024-07-22 14:40:45 +10:00
Michael Vornovitsky	6db91dfefb	extmod/modbtree: Add checks for already-closed database. ... Fixes use-after-free when accessing the database after it is closed with `btree_close`. `btree_close` always succeeds when called with an already-closed database. The new test checks that operations that access the underlying database (get, set, flush, seq) fail with a `ValueError` when the btree is already closed. It also checks that closing and printing the btree succeed when the btree is already closed. Fixes issue #12543. Signed-off-by: Michael Vornovitsky <michaelvornovitskiy@outlook.com>	2024-07-22 10:42:29 +10:00
Damien George	fce3cea244	tests/extmod: Make get_event_loop tests compatible with CPython 3.12. ... Follow up to 2e852522b1: instead of having .exp files for the get_event_loop tests, tweak them so they are compatible with CPython 3.12. This requires calling `asyncio.set_event_loop()` so there is an active event loop and `asyncio.get_event_loop()` succeeds without a warning. Signed-off-by: Damien George <damien@micropython.org>	2024-07-18 11:08:48 +10:00
Sylvain Zimmer	838794ebcc	extmod/mbedtls: Enable GCM and ECDHE-RSA in common mbedtls config. ... Enable support for cipher suites like TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256, as suggested in https://github.com/micropython/micropython/issues/14204#issuecomment-2024366349 and https://github.com/micropython/micropython/issues/10485#issuecomment-1396426824 Tests have been run on the top 500 domains from moz.com. Without this patch, 155 out of 500 fail to connect because of TLS issues. This patch fixes them all. And it seems all existing mbedtls flags are needed to get good coverage of those top 500 domains. The `ssl_poll.py` test has the cipher bits increased from 512 to 1024 in its test key/cert so that it can work with ECDHE-RSA which is now the chosen cipher. Signed-off-by: Sylvain Zimmer <sylvain@sylvainzimmer.com> Signed-off-by: Damien George <damien@micropython.org>	2024-07-02 13:19:17 +10:00
Damien George	2e852522b1	tests/extmod: Add .exp test files for asyncio.get_event_loop tests. ... And use `asyncio.new_event_loop()` where possible. This change is needed because CPython 3.12 deprecated the `get_event_loop()` function. Signed-off-by: Damien George <damien@micropython.org>	2024-05-27 13:56:55 +10:00
Damien George	1ea06b99d8	tests/extmod: Fix regex strings to be of r"" type. ... Otherwise escape characters like \s and \W won't work correctly. Signed-off-by: Damien George <damien@micropython.org>	2024-05-27 13:56:55 +10:00
Damien George	8692d2602a	extmod/asyncio: Make current_task raise exception when there is no task. ... Matches CPython behaviour. Fixes issue #11530. Signed-off-by: Damien George <damien@micropython.org>	2024-02-28 15:48:51 +11:00
Felix Dörre	8547a78275	extmod/modwebsocket: Fix websocket to send correct close frame. ... When the websocket closes currently, it does not send a proper "close"-frame, but rather encodes the 0x8800-sequence inside a binary packet, which is wrong. The close packet is a different kind of websocket frame, according to https://www.rfc-editor.org/rfc/rfc6455. This change resolves an error in Firefox when the websocket closes. Signed-off-by: Felix Dörre <felix@dogcraft.de>	2024-02-26 12:20:40 +11:00
Damien George	b87bbaeb43	tests: Use vfs module instead of os. ... Signed-off-by: Damien George <damien@micropython.org>	2024-02-07 13:25:09 +11:00
Felix Dörre	b802f0f8ab	extmod/modtls: Move the native ssl module to tls. ... The current `ssl` module has quite a few differences to the CPython implementation. This change moves the MicroPython variant to a new `tls` module and provides a wrapper module for `ssl` (in micropython-lib). Users who only rely on implemented comparible behavior can continue to use `ssl`, while users that rely on non-compatible behavior should switch to `tls`. Then we can make the facade in `ssl` more strictly adhere to CPython. Signed-off-by: Felix Dörre <felix@dogcraft.de>	2024-02-07 12:58:52 +11:00
Damien George	d5b96813dc	extmod/modssl_mbedtls: Fix cipher iteration in SSLContext.get_ciphers. ... Prior to this commit it would skip every second cipher returned from mbedtls. The corresponding test is also updated and now passes on esp32, rp2, stm32 and unix. Signed-off-by: Damien George <damien@micropython.org>	2024-01-29 15:11:46 +11:00
Damien George	40687451bb	tests/extmod/framebuf_polygon.py: Replace sys.stdout.write with print. ... So the test doesn't depend on the `sys` module. Signed-off-by: Damien George <damien@micropython.org>	2024-01-29 13:07:25 +11:00
Damien George	6bb446b7ff	tests/extmod: Remove asyncio .exp files that match CPython output. ... These were added back in commit c4935f3049 because the tests required CPython 3.8, which was quite new at the time. But CPython 3.8 was released over 4 years ago (October 2019) and the CI test runners, and developers, have this (or a more recent) CPython version. Removing the .exp files also helps keep MicroPython semantics the same as CPython. The asyncio_fair.py test it adjusted slightly to have more deterministic timing and output. Signed-off-by: Damien George <damien@micropython.org>	2024-01-24 11:20:06 +11:00
Damien George	c3ca3612d1	tests/extmod/asyncio_wait_task.py: Add test for raise and delayed wait. ... This case was fixed in 2ecbad4e91, which stored the exception in the task object. Signed-off-by: Damien George <damien@micropython.org>	2024-01-23 11:13:35 +11:00
Damien George	2ecbad4e91	extmod/asyncio: Support gather of tasks that finish early. ... Adds support to asyncio.gather() for the case that one or more (or all) sub-tasks finish and/or raise an exception before the gather starts. Signed-off-by: Damien George <damien@micropython.org>	2024-01-22 12:03:21 +11:00
Damien George	51fbec2780	tests/extmod/machine_i2s_rate.py: Test multiple I2S instances. ... Signed-off-by: Damien George <damien@micropython.org>	2024-01-22 11:49:28 +11:00
robert-hh	c41b421d48	tests/extmod/machine_uart_tx.py: Add a test for timing of UART.flush(). ... Currently only runs on rp2 but could be extended to run on other targets. Signed-off-by: robert-hh <robert@hammelrath.com>	2024-01-16 09:07:03 +11:00
Damien George	ee226a8b43	all: Fix "reuse" and "overridden" spelling mistakes. ... Codespell doesn't pick up "re-used" or "re-uses", and ignores the tests/ directory, so fix these manually. Signed-off-by: Damien George <damien@micropython.org>	2024-01-05 15:08:33 +11:00
Jim Mussared	74fd7b3d32	tools/ci.sh: Set ulimit -n for unix CI. ... Signed-off-by: Jim Mussared <jim.mussared@gmail.com>	2023-12-21 11:05:31 +11:00
Jim Mussared	8b24aa36ba	extmod/modselect: Handle growing the pollfds allocation correctly. ... The poll_obj_t instances have their pollfd field point into this allocation. So if re-allocating results in a move, we need to update the existing poll_obj_t's. Update the test to cover this case. Fixes issue #12887. This work was funded through GitHub Sponsors. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>	2023-12-21 11:04:59 +11:00
Carlosgg	f3f215e9bd	extmod/modssl_mbedtls: Add SSLContext certificate methods. ... This commit adds: 1) Methods to SSLContext class that match CPython signature: - `SSLContext.load_cert_chain(certfile, keyfile)` - `SSLContext.load_verify_locations(cafile=, cadata=)` - `SSLContext.get_ciphers()` --> ["CIPHERSUITE"] - `SSLContext.set_ciphers(["CIPHERSUITE"])` 2) `sslsocket.cipher()` to get current ciphersuite and protocol version. 3) `ssl.MBEDTLS_VERSION` string constant. 4) Certificate verification errors info instead of `MBEDTLS_ERR_X509_CERT_VERIFY_FAILED`. 5) Tests in `net_inet` and `multi_net` to test these new methods. `SSLContext.load_cert_chain` method allows loading key and cert from disk passing a filepath in `certfile` or `keyfile` options. `SSLContext.load_verify_locations`'s `cafile` option enables the same functionality for ca files. Signed-off-by: Carlos Gil <carlosgilglez@gmail.com>	2023-12-12 16:25:07 +11:00
Damien George	c554df57f6	tests/extmod/deflate_compress.py: Add a test for optimal compression. ... Signed-off-by: Damien George <damien@micropython.org>	2023-11-30 12:13:29 +11:00
Andrew Leech	4cf741062b	extmod/vfs_reader: Add file ioctl to set read buffer size. ... Can be used to speed up importing a file from a vfs based filesystem. Signed-off-by: Andrew Leech <andrew.leech@planetinnovation.com.au>	2023-11-09 11:20:31 +11:00
Jim Mussared	78f4f30cb1	tests/extmod/asyncio_as_uasyncio.py: Fix qstr order dependency. ... This test depends on the order in which qstrs are stored in ROM, which affects the order in which `dir()` will probe the object to see what it supports. Because of the lazy-loading in asyncio/__init__.py, if it tries to do e.g. `wait_for_ms` before `funcs` then it will import funcs, making `funcs` later succeed. But in the other way around, `funcs` will initially not be found. This work was funded through GitHub Sponsors. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>	2023-10-30 11:10:02 +11:00
Christian Walther	7be16e0540	extmod/vfs_posix: Additional tests for coverage of error cases. ... Signed-off-by: Christian Walther <cwalther@gmx.ch>	2023-10-19 16:21:09 +02:00
Christian Walther	be28829ae8	extmod/vfs_posix: Fix getcwd() on non-root VFS. ... The unwritten API contract expected of a VFS.getcwd() by mp_vfs_getcwd() is that its return value should be either "" or "/" when the CWD is at the root of the VFS and otherwise start with a slash and not end with a slash. This was not correctly implemented in VfsPosix for instances with a non-empty root - the required leading slash, if any, was cut off because the root length includes a trailing slash. This would result in missing slashes in the middle of the return value of os.getcwd() or in uninitialized garbage from beyond a string's null terminator when the CWD was at the VFS root. Signed-off-by: Christian Walther <cwalther@gmx.ch>	2023-10-19 16:21:09 +02:00
Christian Walther	0c4fb16871	extmod/vfs_posix: Fix relative paths on non-root VFS. ... The unwritten API contract expected of a VFS by mp_vfs_lookup_path() is that paths passed in are relative to the root of the VFS if they start with '/' and relative to the current directory of the VFS otherwise. This was not correctly implemented in VfsPosix for instances with a non-empty root - all paths were interpreted relative to the root. Fix that. Since VfsPosix tracks its CWD using the "external" CWD of the Unix process, the correct handling for relative paths is to pass them through unmodified. Also, when concatenating absolute paths, fix an off-by-one resulting in a harmless double slash (the root path already has a trailing slash). Signed-off-by: Christian Walther <cwalther@gmx.ch>	2023-10-19 16:21:09 +02:00
Christian Walther	5f7065f57a	extmod/vfs_posix: Fix accidentally passing tests. ... These tests test an unrealistic situation and only pass by accident due to a bug. The upcoming fix for the bug would make them fail. The unrealistic situation is that VfsPosix methods are called with relative paths while the current working directory is somewhere outside of the root of the VFS. In the intended use of VFS objects via os.mount() (as opposed to calling methods directly as the tests do), this never happens, as mp_vfs_lookup_path() directs incoming calls to the VFS that contains the CWD. Make the testing situation realistic by changing the working directory to the root of the VFS before calling methods on it, as the subsequent relative path accesses expect. Thanks to the preceding commit, the tests still pass, but still for the wrong reason. The following commit "Fix relative paths on non-root VFS" will make them pass for the correct reason. Signed-off-by: Christian Walther <cwalther@gmx.ch>	2023-10-19 16:21:08 +02:00
Christian Walther	e3ba6f952b	extmod/vfs_posix: Fix relative root path. ... A VfsPosix created with a relative root path would get confused when chdir() was called on it and become unable to properly resolve absolute paths, because changing directories effectively shifted its root. The simplest fix for that would be to say "don't do that", but since the unit tests themselves do it, fix it by making a relative path absolute before storing it. Signed-off-by: Christian Walther <cwalther@gmx.ch>	2023-10-19 16:21:08 +02:00
Jim Mussared	8eb7721b4a	extmod/modframebuf: Remove FrameBuffer1 from natmod build. ... This work was funded through GitHub Sponsors. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>	2023-10-16 12:18:43 +11:00
Jim Mussared	03a9fa227d	extmod/modframebuf: Fix FrameBuffer get-buffer implementation. ... This wasn't correctly accounting for the bits-per-pixel and was returning a bufinfo struct with the incorrect length. Instead, just forward directly to the underlying buffer object. Fixes issue #12563. This work was funded through GitHub Sponsors. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>	2023-10-16 12:18:08 +11:00
Jim Mussared	d040478d8a	extmod/modframebuf: Validate FrameBuffer bounds against input buffer. ... This ensures that the buffer is large enough for the specified width, height, bits-per-pixel, and stride. Also makes the legacy FrameBuffer1 constructor re-use the FrameBuffer make_new to save some code size. Fixes issue #12562. This work was funded through GitHub Sponsors. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>	2023-10-16 12:17:51 +11:00
Damien George	05cb1406ad	extmod/moductypes: Validate that uctypes.struct addr argument is an int. ... Fixes issue #12660. Signed-off-by: Damien George <damien@micropython.org>	2023-10-12 15:47:15 +11:00
Jim Mussared	fae83a6b4d	tests/extmod/asyncio_threadsafeflag.py: Update for unix select. ... 1. Remove the skip for detecting support for polling user-defined objects as this is always possible now on all ports. 2. Don't print when the scheduled task runs as the ordering of this relative to the other prints is dependent on other factors (e.g. if using the native emitter). This work was funded through GitHub Sponsors. Signed-off-by: Jim Mussared <jim.mussared@gmail.com>	2023-09-29 17:58:40 +10:00
Damien George	62c3033ba6	tests/extmod/vfs_fat_finaliser.py: Tweak test so files are collected. ... Signed-off-by: Damien George <damien@micropython.org>	2023-09-29 12:02:36 +10:00
Damien George	a7e2a6d9f2	tests/extmod/ssl_cadata.py: Skip test on axtls. ... The axtls bindings don't support this. Signed-off-by: Damien George <damien@micropython.org>	2023-09-29 12:01:50 +10:00
Damien George	a33766880e	tests/extmod/deflate_decompress.py: Skip test when not enough memory. ... Signed-off-by: Damien George <damien@micropython.org>	2023-09-29 12:01:04 +10:00
Damien George	397697aa83	py/persistentcode: Always close reader even if an exception is raised. ... Fixes issue #3874. Signed-off-by: Damien George <damien@micropython.org>	2023-09-14 13:03:31 +10:00
Damien George	5e122b11ea	py/parse: Always free lexer even if an exception is raised. ... Fixes issue #3843. Signed-off-by: Damien George <damien@micropython.org>	2023-09-14 13:03:31 +10:00
Damien George	88518009ce	tests/extmod/select_poll_eintr.py: Improve robustness of test. ... Increase allowed range of dt_ms, and print it in case of failure. Signed-off-by: Damien George <damien@micropython.org>	2023-08-15 00:08:51 +10:00