Pablo Andres Fuente

Results 65 comments of Pablo Andres Fuente

Fix `TreeItem` `get_prev*` methods when `p_wrap` is `true`

I didn't add it because I wasn't able to find [Jesusemora](https://github.com/Jesusemora) email. But the link that you shared mentions something about `no-reply` email. I'll try to use it.

Fix `TreeItem` `get_prev*` methods when `p_wrap` is `true`

Done!!!

Fix `TreeItem` `get_prev*` methods when `p_wrap` is `true`

Friendly remainder

Fix `TreeItem` `get_prev*` methods when `p_wrap` is `true`

@Repiteo https://github.com/godotengine/godot/pull/85109 should also be closed

Add unit tests for `String::parse_url()`

I'm not sure if the process of submitting a PR involves saying that it's ready for review, but just in case, I'm saying it.

Add a unit test for `HTTPRequest`

I know the test is really silly and not add too much value, but as I mentioned the idea was to trigger the discussion around FakeIt and HTTPRequest constructor change....

Add a unit test for `HTTPRequest`

Regarding extending the class to mock it, I did that in the past and it ended up being me writing more code on the mock than in actual tests. Using...

Add a unit test for `HTTPRequest`

> EDIT: See [tests/core/test_crypto.h](https://github.com/godotengine/godot/blob/master/tests/core/test_crypto.h) for an example where we use a mock class and all the checks are in the test cases (and not in the mock class as you...

Add a unit test for `HTTPRequest`

Well, this time I decided to change to a simpler Mocking library and I added some new tests that uses it. Of course those tests are quite simple, but it's...

Add a unit test for `HTTPRequest`

I manged to add some more tests and get green on all checks. Can I get a new review? As I mentioned, after I got this merge l will add...