niclabs/two

fix test_event_listen_no_sockets_available on test_event.c

todo opened this issue · 1 comments

todo commented

two/tests/test_event.c

Lines 552 to 556 in 5e762c9

//TODO: fix test_event_listen_no_sockets_available on test_event.c
//UNIT_TEST(test_event_listen_no_sockets_available);
UNIT_TEST(test_event_accept);
UNIT_TEST(test_event_read);
UNIT_TEST(test_event_write);


This issue was generated by todo based on a TODO: comment in 5e762c9. It's been assigned to @gnorambuena because they committed the code.
pipex commented

Before fixing this test, it is necessary to define the behavior of event for incoming clients when no more memory is available.

  • Should we reject immediately
  • Should we let them wait indefinitely (what is being done now)
  • Should we wait some time before rejecting? How much time?