390 Commits

Author	SHA1	Message	Date
Damien George	09ea901317	tests/extmod: Add test to compare time_ns with time. ... They should be close together. Signed-off-by: Damien George <damien@micropython.org>	2024-10-24 23:24:09 +11:00
Damien George	1ec0c9b886	tests/extmod: Use time_ns instead of time in lfs mtime test. ... Because VfsLfs2 uses time_ns to create timestamps for files, and for the test to give consistent results it also needs to use this same function. Signed-off-by: Damien George <damien@micropython.org>	2024-10-24 23:24:09 +11:00
Damien George	ca6aed7649	tests/extmod: Fix access of RTC class in machine.RTC test. ... This previously passed on some targets that automatically import the `machine` module in `boot.py`. Signed-off-by: Damien George <damien@micropython.org>	2024-10-24 16:29:43 +11:00
Damien George	6d98280fda	tests/extmod: Add a simple test for machine.RTC. ... Tests at least that the datetime can be set and get correctly. Signed-off-by: Damien George <damien@micropython.org>	2024-10-24 11:43:03 +11:00
Damien George	3f54e5dff2	tests/extmod: Support esp32,mimxrt,stm32,samd ports in UART TX test. ... Getting this test running on stm32- and mimxrt-based boards requires adding a small delay after constructing the UART so that the initial idle frame has time to be transmitted before the test starts. Also, the timing margin needs to account for an additional 1-bit worth of time on some MCUs. Thanks to @robert-hh for the esp32, mimxrt and samd settings. Signed-off-by: Damien George <damien@micropython.org>	2024-10-22 22:21:17 +11:00
Damien George	0646177997	tests/extmod: Make invalid-blockdev test work consistently on all ports. ... Some ports (eg stm32) configure the FAT driver differently (eg with multi-partition support) and that leads to a slightly different sequence of block reads, compared to other configurations (eg rp2). Comment out the printing in `readblocks()` so the tests are deterministic (the printing is still useful for debugging). Signed-off-by: Damien George <damien@micropython.org>	2024-10-22 14:32:51 +11:00
Damien George	1e3b24238e	tests/extmod: Adjust ssl/tls tests to run on targets with axTLS. ... Signed-off-by: Damien George <damien@micropython.org>	2024-10-22 14:31:13 +11:00
Damien George	7d442373af	tests/extmod: Config SPI test for esp8266 and skip SoftTimer test. ... Signed-off-by: Damien George <damien@micropython.org>	2024-10-22 14:31:13 +11:00
Corran Webster	d1574de3b6	extmod/modframebuf: Fix FrameBuffer size check for stride corner-cases. ... This is a fix for issue #15944, and handles corner cases in the FrameBuffer code when using stride values where the last line's stride may extend past the end of the underlying buffer. This commit includes extra tests for these corner cases. For example a GS8 format FrameBuffer with a width of 8, height of 2 and stride of 10 should be able to fit into a buffer of size 18 (10 bytes for the first horizontal line, and 8 bytes for the second -- the full 10 bytes are not needed). Similarly a 1 by 9 FrameBuffer in MONO_VLSB format with a stride of 10 should be able to fit into a buffer of length 11 (10 bytes for the first 8 lines, and then one byte for the 9th line. Being able to do this is particularly important when cropping the corner of an existing FrameBuffer, either to copy a sprite or to clip drawing. Signed-off-by: Corran Webster <cwebster@unital.dev>	2024-10-22 13:05:31 +11:00
Damien George	97af1001ae	rp2/machine_uart: Make it so TX is done only when no longer busy. ... Prior to this commit, when flushing a UART on the rp2 port, it returns just before the last character is sent out the wire. Fix this by waiting until the BUSY flag is cleared. This also fixes the behaviour of `UART.txdone()` to return `True` only when the last byte has gone out. Updated docs and tests to match. The test now checks that UART TX time is very close to the expected time (prior, it was just testing that the TX time was less than the expected time). Signed-off-by: Damien George <damien@micropython.org>	2024-10-22 10:17:05 +11:00
Angus Gratton	05ac69329d	esp32: Fix hang in taskYIELD() on riscv CPUs when IRQs disabled. ... Regression introduced in 337742f. The hang occurs because the esp32 port was calling "from ISR" port-layer functions to set/clear the interrupt mask. FreeRTOS kernel therefore doesn't know the CPU is in a critical section. In taskYIELD() the riscv port layer blocks after yielding until it knows the yield has happened, and would block indefinitely if IRQs are disabled (until INT WDT triggers). Moving to the "public" portENTER_CRITICAL/portEXIT_CRITICAL API means that FreeRTOS knows we're in a critical section and can react accordingly. Adds a regression test for this case (should be safe to run on all ports). On single core CPUs, this should result in almost exactly the same behaviour apart from fixing this case. On dual core CPUs, we now have cross-CPU mutual exclusion for atomic sections. This also shouldn't change anything, mostly because all the code which enters an atomic section runs on the same CPU. If it does change something, it will be to fix a thread safety bug. There is some risk that this change triggers a FreeRTOS crash where there is a call to a blocking FreeRTOS API with interrupts disabled. Previously this code might have worked, but was probably thread unsafe and would have hung in some circumstances. This work was funded through GitHub Sponsors. Signed-off-by: Angus Gratton <angus@redyak.com.au>	2024-10-10 10:59:51 +11:00
Damien George	197becbdcc	tests: Tweak machine SPI and UART tests to work with esp32c6. ... Signed-off-by: Damien George <damien@micropython.org>	2024-10-09 23:34:27 +11:00
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