bug fix in --event code

- calling read() twice crashed program, now fixed
- added test case to test-event.sh to test with 3 JSON files
- see Issue 8go#60

Author: 8go

.gitignore

@@ -6,12 +6,11 @@ credentials.json.*
 
 # Also do not include the media directory.
 /media/
+/media.*/
 
 # ignore backups
 README.md.*
 matrix-commander.py.*
 
 # ignore help file
 help.txt
-
-# ignore test directory? /test/

README.md

@@ -331,7 +331,7 @@ $ TARGET_EVENT="\$...a.valid.event.id" # event to which to react
 $ REACT_EMOJI="😀" # how to react
 $ printf "$JSON_REACT_MSC2677" "$TARGET_EVENT" "$REACT_EMOJI" |
     matrix-commander.py --event -
-$ # for more examples of "matrix-commander.py --event" see test/test-event.sh
+$ # for more examples of "matrix-commander.py --event" see tests/test-event.sh
 ```
 
 # Usage
@@ -709,7 +709,7 @@ optional arguments:
                         information program will continue to run. This is
                         useful for having version number in the log files.
 
-You are running version 2022-05-23. Enjoy, star on Github and contribute by
+You are running version 2022-05-24. Enjoy, star on Github and contribute by
 submitting a Pull Request.
 ```
 
@@ -804,7 +804,8 @@ See [GPL3 at FSF](https://www.fsf.org/licensing/).
 
 - Thanks to all of you who already have contributed! So appreciated!
   - :heart: and :thumbsup: to @fyfe, @berlincount, @ezwen, @Scriptkiddi,
-    @pelzvieh, @mizlan, @edwinsage, @jschwartzentruber, @nirgal, @benneti, etc.
+    @pelzvieh, @mizlan, @edwinsage, @jschwartzentruber, @nirgal, @benneti,
+    @opk12, etc.
 - Enjoy!
 - Pull requests are welcome  :heart:
 

matrix-commander.py

@@ -338,7 +338,7 @@
 $ REACT_EMOJI="😀" # how to react
 $ printf "$JSON_REACT_MSC2677" "$TARGET_EVENT" "$REACT_EMOJI" |
     matrix-commander.py --event -
-$ # for more examples of "matrix-commander.py --event" see test/test-event.sh
+$ # for more examples of "matrix-commander.py --event" see tests/test-event.sh
 ```
 
 # Usage
@@ -716,7 +716,7 @@
                         information program will continue to run. This is
                         useful for having version number in the log files.
 
-You are running version 2022-05-23. Enjoy, star on Github and contribute by
+You are running version 2022-05-24. Enjoy, star on Github and contribute by
 submitting a Pull Request.
 ```
 
@@ -811,7 +811,8 @@
 
 - Thanks to all of you who already have contributed! So appreciated!
   - :heart: and :thumbsup: to @fyfe, @berlincount, @ezwen, @Scriptkiddi,
-    @pelzvieh, @mizlan, @edwinsage, @jschwartzentruber, @nirgal, @benneti, etc.
+    @pelzvieh, @mizlan, @edwinsage, @jschwartzentruber, @nirgal, @benneti,
+    @opk12, etc.
 - Enjoy!
 - Pull requests are welcome  :heart:
 
@@ -914,7 +915,7 @@
     HAVE_NOTIFY = False
 
 # version number
-VERSION = "2022-05-23"
+VERSION = "2022-05-24"
 # matrix-commander
 PROG_WITHOUT_EXT = os.path.splitext(os.path.basename(__file__))[0]
 # matrix-commander.py
@@ -2089,9 +2090,8 @@ async def send_event(client, rooms, event):  # noqa: C901
     if event == "-":  # - means read as pipe from stdin
         jsondata = sys.stdin.buffer.read()
     else:
-        file = open(event, "r").read()
-        jsondata = file.read()
-        file.close()
+        with open(event, "r") as file:
+            jsondata = file.read()
     logger.debug(
         f"{len(jsondata)} bytes of event data read from file {event}."
     )
@@ -4739,6 +4739,12 @@ def initial_check_of_args() -> None:  # noqa: C901
     if not is_download_media_dir_valid():
         sys.exit(1)
     if not are_arg_files_readable():
+        logger.debug(
+            f"{PROG_WITHOUT_EXT} forces an early abort. "
+            "To avoid partial execution, no action has been performed at all. "
+            "Nothing has been sent. Fix your arguments and run the command "
+            "again."
+        )
         sys.exit(1)
     create_pid_file()
 

mc

@@ -0,0 +1 @@
+matrix-commander.py

test/test-event.sh tests/test-event.sh

@@ -25,16 +25,28 @@ else
 fi
 TARGET_EVENT="$MC_EVENT"
 
-# These 3 test cases should pass
+# These 4 test cases should pass
 printf "$JSON_EDIT_MSC2676" "Fallback body $(date +%H:%M:%S)" "Non-fallback body $(date +%H:%M:%S)" "$TARGET_EVENT" |
-    ./matrix-commander.py --event - $MC_OPTIONS
+    matrix-commander.py --event - $MC_OPTIONS
 # https://unicode.org/emoji/charts/full-emoji-list.html
 # emoji: thumbs up: "👍"; heart: "♥"; smiley: "😀"
 printf "$JSON_REACT_MSC2677" "$TARGET_EVENT" "😀" |
-    ./matrix-commander.py --event - $MC_OPTIONS
+    matrix-commander.py --event - $MC_OPTIONS
 printf "$JSON_REPLY_AS_THREAD_MSC3440" "Thread reply $(date +%H:%M:%S)" "$TARGET_EVENT" |
-    ./matrix-commander.py --event - $MC_OPTIONS
+    matrix-commander.py --event - $MC_OPTIONS
+
+printf "$JSON_EDIT_MSC2676" "Fallback body $(date +%H:%M:%S)" "Non-fallback body $(date +%H:%M:%S)" "$TARGET_EVENT" >event1.json
+printf "$JSON_REACT_MSC2677" "$TARGET_EVENT" "👍" >event2.json
+printf "$JSON_REPLY_AS_THREAD_MSC3440" "Thread reply $(date +%H:%M:%S)" "$TARGET_EVENT" >event3.json
+echo "##############################################################"
+echo "##############################################################"
+echo "##############################################################"
+matrix-commander.py --event event1.json event2.json event3.json -m "" $MC_OPTIONS
+rm event1.json event2.json event3.json
 
+echo "##############################################################"
+echo "##############################################################"
+echo "##############################################################"
 #
 
 # These 4 test cases should ***FAIL***. ***INCORRECT*** JSON objects.